Skip to content

[BUG] Location /books conflicts with COPS links using front controller setup #71

@mikespub

Description

@mikespub

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/A

CPU architecture

x86-64

Docker creation

N/A

Container logs

N/A

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    Status

    No status

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions