After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 779311 - View Source not working under Flatpak
View Source not working under Flatpak
Status: RESOLVED OBSOLETE
Product: geary
Classification: Other
Component: client
master
Other Linux
: Normal normal
: 0.14.0
Assigned To: Geary Maintainers
Geary Maintainers
flatpak
Depends on:
Blocks:
 
 
Reported: 2017-02-27 14:02 UTC by Michael Gratton
Modified: 2019-10-24 04:18 UTC
See Also:
GNOME target: ---
GNOME version: ---



Description Michael Gratton 2017-02-27 14:02:35 UTC
The current method used to implement view source isn't working under Flatpak 0.8 - see https://github.com/flatpak/flatpak/issues/592

We need to either see if the issue will be fixed in Flatpak, or if we'll need to work around it ourselves.

Spun off from Bug 770886.
Comment 1 Michael Gratton 2017-09-23 12:12:28 UTC
According to feedback in the Flatpak issue above, this would need some portal support at least, but the current implementation is a bit of a hack anyway.

So we have some options here:

1. Hide View Source menu item if we can detect we are running in a flatpak sandbox
2. Stop using an external and integrate a viewer into Geary, either a) as a popup-window (boo!) or b) into the the conversation view (yay!)
3. Save the file to the XDG Downloads dir and open the file in an external app from there

(3) is the closest to current functionality, but sucks because it will leave view source files littered around people's downloads folder. (1) and (2a) are both quick fixes but (2a) is probably preferable. (2b) is probably the best option, but a lot of work.
Comment 2 Michael Gratton 2017-09-23 12:13:19 UTC
Maybe go with (1) for 0.12.1 and either (2a) or (2b) for 0.13?
Comment 3 Niels De Graef 2017-09-28 10:29:23 UTC
There's maybe also (4) Adding `--filesystem=/tmp` to the Flatpak Manifest.

The privacy implications of storing mail in /tmp aside, this is what we do already, so might as well keep the same functionality for 0.12?
Comment 4 Michael Gratton 2017-09-28 14:55:10 UTC
Oh yeah, good call! Workaround pushed to as commit 919e4b9.

It's still a bit suboptimal, since it now presents the weird-looking portal app-chooser UI rather than just opening it, but at least the feature is back. Will keep this open for now.
Comment 5 Federico Bruni 2017-12-12 07:44:40 UTC
The weird-looking portal app-chooser UI appears also every time you open any attachment.
Comment 6 Michael Gratton 2018-06-26 04:47:56 UTC
Bump tickets to 0.14 that aren't going to make 0.13.
Comment 7 Michael Gratton 2019-10-24 04:18:37 UTC
Closing this in favour of https://gitlab.gnome.org/GNOME/geary/issues/453