webbrowser-app

Merge lp://qastaging/~ahayzen/webbrowser-app/fix-1620635-apparmor-download-rule into lp://qastaging/webbrowser-app/staging

  1. fix-1620635-apparmor-download-rule
  2. Merge into staging
Proposed by Andrew Hayzen
Status: Merged
Merged at revision: 1637
Proposed branch: lp://qastaging/~ahayzen/webbrowser-app/fix-1620635-apparmor-download-rule
Merge into: lp://qastaging/webbrowser-app/staging
Diff against target: 25 lines (+2/-2)
2 files modified
debian/rules (+2/-1)
debian/webbrowser-app-apparmor.manifest (+0/-1)
To merge this branch: bzr merge lp://qastaging/~ahayzen/webbrowser-app/fix-1620635-apparmor-download-rule
Reviewer Review Type Date Requested Status
Jamie Strandboge Approve
Review via email: mp+320521@code.qastaging.launchpad.net

Commit message

* Add "@{HOME}/@{XDG_DOWNLOAD_DIR}/** rwk," manually rather than via write_paths as we don't want an owner. Due to bug 1620635 - content-hub uses libapparmor to check the read path and this fails with rules containing owner

Description of the change

* Add "@{HOME}/@{XDG_DOWNLOAD_DIR}/** rwk," manually rather than via write_paths as we don't want an owner. Due to bug 1620635 - content-hub uses libapparmor to check the read path and this fails with rules containing owner

To post a comment you must log in.
Revision history for this message
Jamie Strandboge (jdstrand) wrote :

This looks fine to me and works around the aforementioned bug.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches