Skip to content

Correct filter name from PreserveHostHeader to preserveHost#4211

Closed
think-gem wants to merge 1 commit into
spring-cloud:mainfrom
think-gem:patch-1
Closed

Correct filter name from PreserveHostHeader to preserveHost#4211
think-gem wants to merge 1 commit into
spring-cloud:mainfrom
think-gem:patch-1

Conversation

@think-gem

Copy link
Copy Markdown

In the WebMVC version of Spring Cloud Gateway, the global defaultFilters property (available in WebFlux) is missing. I hope the official team can add this feature. Currently, if I add filters to each route, the filter name for the route is preserveHost instead of PreserveHostHeader.

In the WebMVC version of Spring Cloud Gateway, the global defaultFilters property (available in WebFlux) is missing. I hope the official team can add this feature. Currently, if I add filters to each route, the filter name for the route is preserveHost instead of PreserveHostHeader.

Signed-off-by: thinkgem <thinkgem@163.com>
@ryanjbaxter

Copy link
Copy Markdown
Contributor

Please open a separate issue about the defaultFilters.

I think the right way to address this PreserveHostHeader issue is to make the name match WebFlux. I have opened a PR here for this: #4213

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants