Skip to content
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

OPML export fails/complicated due to SailJail #209

Open
RaphaelWimmer opened this issue Oct 20, 2024 · 5 comments · May be fixed by #213
Open

OPML export fails/complicated due to SailJail #209

RaphaelWimmer opened this issue Oct 20, 2024 · 5 comments · May be fixed by #213
Assignees

Comments

@RaphaelWimmer
Copy link

Current Sailfish OS versions enforce sailjail/firejail rules.
When one tries to export the list of podcasts into an OPML file, gpodder asks for a filename (i.e. does not present a folder selection screen).
The file is then saved into the user's home directory with a ".opml" extension.
However, the exported file never reaches the actual home directory but stays in the sailjail/firejail container and is lost when gpodder is closed.
As far as I understand the issue, gpodder would either

  • a) need a SailJail profile with an explicit permission to access the home directory or (preferrable)
  • b) need to export to one of the directories listed in /etc/sailjail/permissions/UserDirs.permission (e.g., Documents, Downloads, or Music) or to the SD card

Workaround: enter Documents/export.opml in the file name field when exporting the podcasts.

Alternative workaround (tried this first before looking into permissions; adjust the path to only get the .opml file you want):

PID=$(firejail --list | grep gpodder | cut -d: -f 1 )
firejail --cat=$PID /home/*/*.opml > /tmp/podcasts.opml
RaphaelWimmer added a commit to RaphaelWimmer/gpodder-sailfish that referenced this issue Oct 20, 2024
@RaphaelWimmer RaphaelWimmer changed the title OPML export limited due to Sailjail OPML export fails/complicated due to SailJail Oct 20, 2024
@Keeper-of-the-Keys
Copy link
Contributor

Interesting, I guess because I have it installed over many versions I have more permissions grandfathered in, adding a Sailjail profile has been on my list of things I want to get to for a while, just didn't have priority since things seemed to be working regardless.

Can you explain a bit more why you think b is preferable to a?

@Keeper-of-the-Keys
Copy link
Contributor

My view is that a is preferable since I also want to allow people to set their own custom download location and I am not aware of SFOS/Sailjail prompting the user for additional permissions only when needed.

@RaphaelWimmer
Copy link
Author

My view is that a is preferable since I also want to allow people to set their own custom download location and I am not aware of SFOS/Sailjail prompting the user for additional permissions only when needed.

Yes, you are right. Option b) seemed just to be the easier option.

Keeper-of-the-Keys pushed a commit to Keeper-of-the-Keys/gpodder-sailfish that referenced this issue Nov 25, 2024
- Audio
- UserDirs - since we allow the user to change the Downloads directory
- RemovableMedia - same as UserDirs
- Internet

It is unclear to me if UserDirs also includes ~, if not we will need to update the export code to fix gpodder#209

Signed-off-by: E.S. Rosenberg a.k.a. Keeper of the Keys <[email protected]>
@Keeper-of-the-Keys
Copy link
Contributor

FYI - It turns out that it has to be a combination of your fix and my fix because with SailJail ${HOME} is not accessible, I hope to merge soon and release a very long overdue release full of fixes.

@Keeper-of-the-Keys Keeper-of-the-Keys linked a pull request Dec 8, 2024 that will close this issue
@RaphaelWimmer
Copy link
Author

Thanks for the update! I'm also following your SailJail journey on the SFOS forums (link to improve discoverability for others having fun with SailJail).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment