search

OCA / storage

GNU Affero General Public License v3.0
70 stars 159 forks source link

[16.0][FIX] fs_storage: add missing sudo when creating FilsSystem #385

Closed sbidoul closed 2 months ago

sbidoul commented 2 months ago

This is necessary for non admin users to access file systems, for instance for attachments.

This went unnoticed so far presumably because the filesystem is cached and likely accessed by the server during server bootstrap. I could confirm this by starting my dev server, accessing an attachment as a normal user (failed), then accessing an attachment as admin (succeeded), then as a normal user again (success).

lmignon commented 2 months ago

/ocabot merge patch

OCA-git-bot commented 2 months ago

On my way to merge this fine PR! Prepared branch 16.0-ocabot-merge-pr-385-by-lmignon-bump-patch, awaiting test results.

OCA-git-bot commented 2 months ago

Congratulations, your PR was merged at f4689c64c03333077e0af79ea79790f284b7a0e6. Thanks a lot for contributing to OCA. ❤️