Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[11.x] Restore Request::HEADER_X_FORWARDED_PREFIX in TrustProxies #52598

Merged
merged 2 commits into from
Aug 31, 2024

Conversation

taka-oyama
Copy link
Contributor

@taka-oyama taka-oyama commented Aug 29, 2024

It seems like Request::HEADER_X_FORWARDED_PREFIX was removed in #47309 but we need this header to make this work in our environment.

Looking at the actual diff, it seems like it was removed by accident?
There was not explanation for it being removed at least.

This fix will also make it aligned with the default fallback when $header is not defined.

@taka-oyama taka-oyama changed the title [12.x] Restore Request::HEADER_X_FORWARDED_PREFIX in TrustProxies [11.x] Restore Request::HEADER_X_FORWARDED_PREFIX in TrustProxies Aug 29, 2024
@taylorotwell taylorotwell merged commit d343e5b into laravel:11.x Aug 31, 2024
29 checks passed
@taka-oyama taka-oyama deleted the fix/missing-header-prefix branch September 1, 2024 00:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants