Is there an existing issue for this?
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/A
CPU architecture
x86-64
Docker creation
Container logs
Is there an existing issue for this?
Current Behavior
The nginx default.conf.sample uses
Location /booksfor 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
CPU architecture
x86-64
Docker creation
Container logs