Skip to content

Optionally enable REQUEST_URI rewrites #1615

@scheibling

Description

@scheibling

Feature Overview

Just to start off, a massive thanks to you all for your work on this. We use it enthusiastically for a number of things, and we appreciate you all!

With a recent update of one of our custom production builds of this (alpine + custom PHP image), we noticed that the rewrites for the REQUEST_URI (87077ec) were disabled.

Would it be an option to make it configurable either on compile or settings-wise?

Alternatives Considered

The adding of custom (dynamic) request headers to the backend would be an alternative, to be able to send something like X_REWRITTEN_REQUEST_URI would maybe align better with the roadmap and be more generally useful

Adding a second url_rewrite/uri_rewrite field with the old functionality maybe is probably the easiest alternative

Additional Context

Even though this is not strictly necessary, and probably not useful for 90% of users we have some edge cases where we'd like it to work.

With that said - at this point this is mostly a question of if any of these alternatives would be acceptable in terms of direction. I would be happy to find the time to work/help work on a PR for an implementation of one of them.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions