-
-
Notifications
You must be signed in to change notification settings - Fork 11
Description
Is there an existing issue for this?
- I have searched the existing issues
Current Behavior
The nginx default.conf.sample uses Location /books for internal X-Accel-Redirect links.
This overlaps with standard COPS links for book details when using a front controller setup (i.e. no index.php/ in links).
The default nginx config doesn't support front controller setup by default either, so users with default configs won't notice this at the moment.
COPS 4.2+ supports mapping of X-Accel-Redirect filepaths in config/local.php, so both issues can be corrected by using a new internal /_redirect_ location and adapting the default config files.
A new PR will be prepared when COPS 4.2.0 is released and pending #70 has been merged to avoid conflict.
Expected Behavior
Enable front controller and define mapping in COPS config/local.php and adapt nginx default.conf.sample to support both front controller setup and new internal redirect location
Steps To Reproduce
N/A
Environment
- OS:N/A
- How docker service was installed:N/ACPU architecture
x86-64
Docker creation
N/AContainer logs
N/AMetadata
Metadata
Assignees
Labels
Type
Projects
Status