TMSU icon indicating copy to clipboard operation
TMSU copied to clipboard

Provide a DB setting to accept only relative paths (paths within the TMSU root)

Open ipkiss42 opened this issue 3 years ago • 2 comments

As a user, I would like my folders with a TMSU database to be relocatable, i.e. I don't want to store in the DB a file which "escapes" the root folder of the TMSU database. Since TMSU already stores relative paths whenever possible, we are almost there, but I can still tag an absolute path by mistake, or tag a symlink pointing outside of the root, or even be affected by #168.

It would be nice if TMSU allowed me to avoid mistakes by rejecting explicitly paths outside of the root, with a setting in the DB.

This setting might even become the default for new databases. On a typical system, I suspect that most databases would be "relative-only", except one, used as a fallback.

ipkiss42 avatar Sep 21 '20 13:09 ipkiss42

No, it's not always possible or desirable to have the database collocated with the files. Think read-only media, network files, stuff on slow access tape drives &c.

On Mon, 21 Sep 2020, 14:04 Olivier Teulière, [email protected] wrote:

As a user, I would like my folders with a TMSU database to be relocatable, i.e. I don't want to store in the DB a file which "escapes" the root folder of the TMSU database. Since TMSU already stores relative paths whenever possible, we are almost there, but I can still tag an absolute path by mistake, or tag a symlink pointing outside of the root, or even be affected by #168 https://github.com/oniony/TMSU/issues/168.

It would be nice if TMSU allowed me to avoid mistakes by rejecting explicitly paths outside of the root, with a setting in the DB.

This setting might even become the default for new databases. On a typical system, I suspect that most databases would be "relative-only", except one, used as a fallback.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/oniony/TMSU/issues/207, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAABOBYYE5CVSRTPJG4IC5TSG5FL7ANCNFSM4RUQHUVA .

oniony avatar Sep 21 '20 13:09 oniony

OK, then it shouldn't be the default indeed.

ipkiss42 avatar Sep 21 '20 14:09 ipkiss42