Closed (fixed)
Project:
Drupal core
Version:
8.6.x-dev
Component:
Umami demo
Priority:
Major
Category:
Bug report
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
1 Feb 2018 at 22:51 UTC
Updated:
23 Feb 2018 at 16:59 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
larowlanthis should work, but I use nginx with fpm, so can't be sure the pass is actually a pass
Comment #3
eli-tTested 2941488-htaccess-pass.patch with Apache - with the patch applied, attempting to load core/profiles/demo_umami/modules/demo_umami_content/default_content/images/chocolate-brownie-umami.jpg results in a 403 response. Without the patch, the image is loaded.
htaccess file looks fine.
Comment #5
alexpott@Eli-T thanks for documenting the steps you took to test the patch - adding review credit.
Comment #6
alexpottCommitted 9c78355 and pushed to 8.6.x. Thanks!
Comment #8
alexpottRe nginx. I guess nginx and other webservers will still have this issue. If we backport umami to 8.5.0 then we might want to mention this somewhere deep in the release notes. A more general solution for this problem is #2936841: Remove images from demo_umami profile and download upon installation instead
Comment #9
eli-t@alexpott wrt #2936841: Remove images from demo_umami profile and download upon installation instead, it's possible that we might keep one image to use as a fallback when no network is available when installing the Umami profile, so that issue might not mititgate the requirement for htaccess.
Also #2940362: Move recipe instruction out of csv file in to their own text file. proposes adding another htaccess file to the folder containing the html files with recipe instructions.