mirror of
https://github.com/netblue30/firejail.git
synced 2026-05-15 14:16:14 -06:00
[GH-ISSUE #2361] "desktop.profile" #1573
Labels
No labels
LTS merge
LTS merge
bug
bug
converted-to-discussion
doc-todo
documentation
duplicate
enhancement
file-transfer
firecfg
firejail-in-firejail
firetools
graphics
help wanted
information_old
installation
invalid
modif
moved
needinfo
networking
notabug
notourbug
old-version
overlayfs
packaging
profile-request
pull-request
question
question_old
removal
runtime-permissions
sandbox-ipc
security
stale
wiki
wiki
wontfix
wordpress
workaround
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference: github-starred/firejail#1573
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @reinerh on GitHub (Jan 21, 2019).
Original GitHub issue: https://github.com/netblue30/firejail/issues/2361
I noticed a profile called "desktop.profile".
Is this named correctly? The local file it includes is actually named "github-desktop.local", so it suggests that "github-desktop.profile" would be the correct name.
It would also be quite strange if the github-desktop binary was just called "desktop". But I can't check it at the moment.
Can someone please verify if the name is correct?
cc @glitsj16
@SkewedZeppelin commented on GitHub (Jan 21, 2019):
this is https://github.com/desktop/desktop
there appear to be no official binaries for Linux, nor do any distros seem to package it
there are a few unofficial builds of it, one I checked and the binary was
github-desktopsince there isn't a github-desktop.profile I'm going to assume the file name was an accident, but lets wait for @glitsj16's response
@ghost commented on GitHub (Jan 21, 2019):
Hi, I remember this application going through some (file)name changes. At the time I added the profile the binary was named
desktop, indeed strange and exceptional. For a while now it has been available in the AUR in a few differently named packages for Arch Linux. To confuse things even more, there's https://github.com/shiftkey/desktop and https://snapcraft.io/github-desktop, offering Appimage, Snap, .deb and .rpm's.@reinerh: You are correct though. The profile should be named
github-desktop.profile. On top of that, the aforementioned installers now all use the more sensiblegithub-desktop.desktopformat. Hence the firecfg name also needs changing. If you and @SkewedZeppelin are okay with this, I'll make a PR with the needed changes asap.Thanks for pointing out this mistake, which was mine indeed.
@ghost commented on GitHub (Jan 21, 2019):
Follow-up. I just noticed that the .deb from https://github.com/shiftkey/desktop/releases installs to
/opt/Github Desktop. I've never used anything from /opt with firejail, so give me some time to boot into Ubuntu 16.04, install the .deb and test an edited profile. I'll report back here, please hold.@reinerh commented on GitHub (Jan 21, 2019):
Thank you for looking into it. 👍
@ghost commented on GitHub (Jan 22, 2019):
Hi all. After some more testing I came up with the following PR: https://github.com/netblue30/firejail/pull/2362. Please check and/or advise on these changes. I added comments inside the profile to cover 2 possible issues:
@ghost commented on GitHub (Jan 22, 2019):
PR https://github.com/netblue30/firejail/pull/2362 fixes the profile name issue and refactors it as a blacklist profile with additional support for github-desktop appimage.