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.

Sign up for GitHub

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

Jump to bottom

Support X-Forwarded-Prefix in DefaultHttpForwardedHeaderHandler #3432

Open
philwebb opened this issue Sep 16, 2024 · 1 comment · May be fixed by #3436
Open

Support X-Forwarded-Prefix in DefaultHttpForwardedHeaderHandler #3432

philwebb opened this issue Sep 16, 2024 · 1 comment · May be fixed by #3436
Assignees
@chemicL
Labels
type/enhancement A general enhancement
Milestone
1.1.23

Comments

@philwebb
Copy link

This has been raised previously, but I thought it was worth asking again since we had a recent Spring Boot issue raised about it. Although it's possible to write code to support X-Forwarded-Prefix headers, it would be nice if DefaultHttpForwardedHeaderHandler handled them automatically.
@philwebb philwebb added status/need-triage A new issue that still need to be evaluated as a whole type/enhancement A general enhancement labels Sep 16, 2024
@philwebb philwebb mentioned this issue Sep 16, 2024
Closed
@violetagg violetagg self-assigned this Sep 17, 2024
@violetagg violetagg removed the status/need-triage A new issue that still need to be evaluated as a whole label Sep 17, 2024
@chemicL chemicL assigned chemicL and unassigned violetagg Sep 17, 2024
@chemicL
Copy link
Member

chemicL commented Sep 17, 2024

I'll try to have a look at this.

chemicL added a commit that referenced this issue Sep 18, 2024
@chemicL
Parse X-Forwarded-Prefix request header
957375a 
The X-Forwarded-Prefix can be obtained via
`HttpServerRequest#forwardedPrefix()`.

Resolves #3432
@chemicL chemicL linked a pull request Sep 18, 2024 that will close this issue
Parse X-Forwarded-Prefix request header #3436
Open
@violetagg violetagg added this to the 1.1.23 milestone Sep 19, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@chemicL chemicL

Labels
type/enhancement A general enhancement
Projects
None yet
Milestone
1.1.23
Development

Successfully merging a pull request may close this issue.

Parse X-Forwarded-Prefix request header reactor/reactor-netty
3 participants
@philwebb @violetagg @chemicL