- Status changed to Needs review
about 1 year ago 12:31pm 26 May 2023 - First commit to issue fork.
- @afinnarn opened merge request.
- 🇺🇸United States afinnarn
Hello. I looked at the patch and it has "'system/files" hardcoded as the base path for private files. I made a MR that should cover the private scheme as well as the public scheme.
Please take a look, see if it works for you, and let me know what you think.
- 🇺🇸United States cameron prince
cameron prince → made their first commit to this issue’s fork.
- Status changed to Fixed
8 months ago 7:11pm 17 October 2023 - 🇺🇸United States cameron prince
Changes look good and have been merged. Closing this as fixed!
Automatically closed - issue fixed for 2 weeks with no activity.
- Status changed to Fixed
4 months ago 10:58am 21 February 2024 - 🇳🇱Netherlands Rhezios
Hi,
Hello. I looked at the patch and it has "'system/files" hardcoded as the base path for private files. I made a MR that should cover the private scheme as well as the public scheme.
The reason for the hardcoded string is the route for private files. Private files are served by the route "system.private_file_download" and not the path of the file itself. PrivateStream::basePath() returns "Settings::get('file_private_path')" which is not accessible over the web server.
* A local file system path where private files will be stored. This directory
* must be absolute, outside of the Drupal installation directory and not
* accessible over the web.