-
Notifications
You must be signed in to change notification settings - Fork 555
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
please add support for subfolders for {PICTURES} and others #2359
Labels
enhancement
New feature request
Comments
kmk3
added a commit
to kmk3/firejail
that referenced
this issue
Feb 9, 2021
This reverts commit 5df1f27. That commit breaks things, as pointed out by @rusty-snake[1]: > @kmk3 @glitsj16 The xdg macros are treated literally if they have sub > components (netblue30#2359): > > ``` > Error: "${DOCUMENTS}/KeePassXC" is an invalid filename: rejected character: "{" > ``` [1]: netblue30@3fa2927#commitcomment-46913219
rusty-snake
pushed a commit
that referenced
this issue
Feb 9, 2021
This reverts commit 5df1f27. That commit breaks things, as pointed out by @rusty-snake[1]: > @kmk3 @glitsj16 The xdg macros are treated literally if they have sub > components (#2359): > > ``` > Error: "${DOCUMENTS}/KeePassXC" is an invalid filename: rejected character: "{" > ``` [1]: 3fa2927#commitcomment-46913219
6 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
hello for flameshot i need to add mkdir
${PICTURES}/Screenshots
but unfortunately subfolders are only supported for ${home}. So with a whitelist profile the program is returning an error because there is no Screenshots folder and it look like he can't create it, you need to create it manually (assuming that you have noblacklisted and whitelisted the whole ${PICTURES}). also for privacy it would be better to allow only a subfolder of Pictures instead of giving access to everything.it would be very useful, thanks.
The text was updated successfully, but these errors were encountered: