Skip to content

Update properties for forwarded headers - #51127

Closed
VedPatel2811 wants to merge 1 commit into
spring-projects:mainfrom
VedPatel2811:gh-51030-forwarded-headers-properties
Closed

Update properties for forwarded headers#51127
VedPatel2811 wants to merge 1 commit into
spring-projects:mainfrom
VedPatel2811:gh-51030-forwarded-headers-properties

Conversation

@VedPatel2811

Copy link
Copy Markdown

Summary

Updates properties for forwarded headers to adapt to upcoming Spring Framework 7.1 changes (spring-projects/spring-framework#37072), which requires an explicit choice of forwarded header types (Forwarded vs X-Forwarded-*).

Changes

  • Added nested Forwarded class and ForwardedHeaderType enum to ServerProperties (server.forwarded.header-type, server.forwarded.x-forwarded-ssl-enabled, server.forwarded.x-forwarded-prefix-enabled).
  • Updated ServletWebServerConfiguration and ReactiveWebServerConfiguration to wire new properties onto ForwardedHeaderFilter and ForwardedHeaderTransformer.
  • Expanded proxy server documentation in webserver.adoc with header-type selection guidance, YAML examples, and security notes.
  • Added corresponding unit tests in servlet, reactive, and Tomcat test suites.

Closes gh-51030

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Jul 25, 2026
@VedPatel2811
VedPatel2811 force-pushed the gh-51030-forwarded-headers-properties branch from b89c0fd to 944b26b Compare July 25, 2026 17:13
Closes spring-projectsgh-51030"

Signed-off-by: Ved Patel <veds28112004@gmail.com>
@snicoll

snicoll commented Jul 25, 2026

Copy link
Copy Markdown
Member

Thanks for the PR but @bclozel was already working on this. I've assigned it on the issue, and sorry for the wasted efforts.

@snicoll snicoll closed this Jul 25, 2026
@snicoll snicoll added status: declined A suggestion or change that we don't feel we should currently apply and removed status: waiting-for-triage An issue we've not yet triaged labels Jul 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status: declined A suggestion or change that we don't feel we should currently apply

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Update properties for forwarded headers to adapt to Spring Framework changes

3 participants