[GH-ISSUE #4706] Implement case insensitive sorting of profiles on GitHub to avoid duplication #2760

Closed
opened 2026-05-05 09:25:19 -06:00 by gitea-mirror · 4 comments
Owner

Originally created by @Lonniebiz on GitHub (Nov 22, 2021).
Original GitHub issue: https://github.com/netblue30/firejail/issues/4706

I downloaded an AppImage of discord from here:
https://www.appimagehub.com/p/1420323/

Debian 11 stable provides firejail version 0.9.64.4, which doesn't have a profile in /etc/firejail/ for discord.

At https://github.com/netblue30/firejail/tree/master/etc/profile-a-l , I located these profile files:

Discord.profile
discord.profile
DiscordCanary.profile
discord-canary.profile
discord-common.profile

Notice that multiple similar files only differ by capitalization. This is probably a product of the poor sort-order offered by github here and such a "cause of redundancy" may be avoided in the future if it is possible to set github to do some type of case insensitive sorting of profiles that would group these files (above) next to each other and prevent people from overlooking when a profile already exists but has different capitalization.

Also, when I look inside these files I see includes to other (non-discord-named) files.

So, far, when I've found profiles on github, since they are not in my /etc/firejail/ already, I've considered them custom and I've been placing them at ~/.config/firejail/ instead of just placing them in the /etc/firejail/ folder that came with my distribution's installation of firejail.

However, in this case, I'm not sure what to do. I could manually follow all includes and put all dependencies files into ~/.config/firejail/, or I could just place the proper discord profiles into /etc/firejail/ where most of those non-discord includes within (like "include globals.local") will likely already reside at a resolvable relative path.

Or, I could just make one file that includes everything instead of including all the separate files in ~/.config/firejail/ .

So, I'm reporting this seeming redundancy located at https://github.com/netblue30/firejail/tree/master/etc/profile-a-l for discord, but I'd appreciate any advise regarding the proper location to start consuming the correct profile for discord.

Originally created by @Lonniebiz on GitHub (Nov 22, 2021). Original GitHub issue: https://github.com/netblue30/firejail/issues/4706 I downloaded an AppImage of discord from here: https://www.appimagehub.com/p/1420323/ Debian 11 stable provides firejail version 0.9.64.4, which doesn't have a profile in /etc/firejail/ for discord. At https://github.com/netblue30/firejail/tree/master/etc/profile-a-l , I located these profile files: Discord.profile discord.profile DiscordCanary.profile discord-canary.profile discord-common.profile Notice that multiple similar files only differ by capitalization. This is probably a product of the poor sort-order offered by github here and such a "cause of redundancy" may be avoided in the future if it is possible to set github to do some type of case insensitive sorting of profiles that would group these files (above) next to each other and prevent people from overlooking when a profile already exists but has different capitalization. Also, when I look inside these files I see includes to other (non-discord-named) files. So, far, when I've found profiles on github, since they are not in my /etc/firejail/ already, I've considered them **custom** and I've been placing them at ~/.config/firejail/ instead of just placing them in the /etc/firejail/ folder that came with my distribution's installation of firejail. However, in this case, I'm not sure what to do. I could manually follow all includes and put all dependencies files into ~/.config/firejail/, or I could just place the proper discord profiles into /etc/firejail/ where most of those non-discord includes within (like "include globals.local") will likely already reside at a resolvable relative path. Or, I could just make one file that includes everything instead of including all the separate files in ~/.config/firejail/ . So, I'm reporting this seeming redundancy located at https://github.com/netblue30/firejail/tree/master/etc/profile-a-l for discord, but I'd appreciate any advise regarding the proper location to start consuming the correct profile for discord.
gitea-mirror 2026-05-05 09:25:19 -06:00
  • closed this issue
  • added the
    notabug
    label
Author
Owner

@rusty-snake commented on GitHub (Nov 22, 2021):

The easiest way to find a (pro)file on github (case insensitive): https://github.com/netblue30/firejail/find/master


Copying profiles from github (master) to an stable release of firejail can cause trouble because they are written for a newer firejail version.

Debian 11 stable provides firejail version 0.9.64.4, which doesn't have a profile in /etc/firejail/ for discord.

um wait a minute, this pages says something else https://packages.debian.org/bullseye/all/firejail-profiles/filelist

<!-- gh-comment-id:975204907 --> @rusty-snake commented on GitHub (Nov 22, 2021): The easiest way to find a (pro)file on github (case insensitive): https://github.com/netblue30/firejail/find/master --- Copying profiles from github (master) to an stable release of firejail can cause trouble because they are written for a newer firejail version. > Debian 11 stable provides firejail version 0.9.64.4, which doesn't have a profile in /etc/firejail/ for discord. um wait a minute, this pages says something else https://packages.debian.org/bullseye/all/firejail-profiles/filelist
Author
Owner

@Lonniebiz commented on GitHub (Nov 22, 2021):

@rusty-snake It took me a moment to realize that I could just type to filter that master list, but I do like the quick-search-profile-link you provided. I'll go there from now on when looking for profiles.

As for discord being in /etc/firejail/, it is indeed there. I did look yesterday and I'm not sure how I overlooked it, but I did. Thank you.

Are the varying capitalizations of the discord profiles intentional?

<!-- gh-comment-id:975659139 --> @Lonniebiz commented on GitHub (Nov 22, 2021): @rusty-snake It took me a moment to realize that I could just type to filter that master list, but I do like the [quick-search-profile-link](https://github.com/netblue30/firejail/find/master) you provided. I'll go there from now on when looking for profiles. As for discord being in `/etc/firejail/`, it is indeed there. I did look yesterday and I'm not sure how I overlooked it, but I did. Thank you. Are the varying capitalizations of the discord profiles intentional?
Author
Owner

@reinerh commented on GitHub (Nov 23, 2021):

Are the varying capitalizations of the discord profiles intentional?

Profile names are capitalized the same as the filenames of the binaries that are distributed (some packages might distribute it as Discord (maybe the official one?), others maybe as discord).
Firejail uses the filename of the application to find the profile that should be applied to it.

<!-- gh-comment-id:977264281 --> @reinerh commented on GitHub (Nov 23, 2021): > Are the varying capitalizations of the discord profiles intentional? Profile names are capitalized the same as the filenames of the binaries that are distributed (some packages might distribute it as `Discord` (maybe the official one?), others maybe as `discord`). Firejail uses the filename of the application to find the profile that should be applied to it.
Author
Owner

@rusty-snake commented on GitHub (Jun 8, 2022):

See @reinerh's comment.

<!-- gh-comment-id:1149797896 --> @rusty-snake commented on GitHub (Jun 8, 2022): See @reinerh's comment.
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: github-starred/firejail#2760
No description provided.