Skip to content

fix(apache): scope NE rewrite flag to API routes - #324

Open
rajanpanth wants to merge 1 commit into
gitlabhq:masterfrom
rajanpanth:fix/304-apache-ne-flag
Open

fix(apache): scope NE rewrite flag to API routes#324
rajanpanth wants to merge 1 commit into
gitlabhq:masterfrom
rajanpanth:fix/304-apache-ne-flag

Conversation

@rajanpanth

Copy link
Copy Markdown

Summary

  • keep NE only on a dedicated API rewrite rule (^/api/v[0-9]+/.*) to preserve encoded API paths
  • remove NE from the general workhorse proxy rewrite so whitespace paths are rewritten safely
  • apply the same change consistently across all Apache 2.2/2.4 + SSL/omnibus recipe variants

Why

Issue #304 reports 400 errors for paths containing whitespace when the global proxy rewrite uses NE.
This restores the earlier split behavior (API paths keep NE, general paths do not), so both use-cases are covered.

Validation

  • deterministic pattern check confirms all 8 Apache recipe files now contain:
    • one API-specific NE rewrite
    • one general non-NE rewrite

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant