-
Notifications
You must be signed in to change notification settings - Fork 3.4k
Support setting Spring.webflux.base-path and supporting it in path predicate #2984
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
Conversation
|
@Guofuyinan Please sign the Contributor License Agreement! Click here to manually synchronize the status of this Pull Request. See the FAQ for frequently asked questions. |
|
@Guofuyinan Thank you for signing the Contributor License Agreement! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please add a test
I will add a test ASAP |
920d2ab to
1ceaee8
Compare
...t/java/org/springframework/cloud/gateway/handler/predicate/GatewayPredicateVisitorTests.java
Show resolved
Hide resolved
...main/java/org/springframework/cloud/gateway/handler/predicate/PathRoutePredicateFactory.java
Outdated
Show resolved
Hide resolved
d748811 to
c967c2c
Compare
|
Looks like some checkstyle issues. |
Try again |
|
@spencergibb any update? |
|
We will get this included in the 2024.0 release train. |
@spencergibb any update? |
|
Fixes #1759 |
|
Extract variables |
|
@ryanjbaxter Any progress or anything I can do? |
...main/java/org/springframework/cloud/gateway/handler/predicate/PathRoutePredicateFactory.java
Outdated
Show resolved
Hide resolved
...main/java/org/springframework/cloud/gateway/handler/predicate/PathRoutePredicateFactory.java
Show resolved
Hide resolved
|
Can you submit this against the 4.2.x branch? |
…edicate Signed-off-by: 郭付一男 <guofuyinan@gmail.com>
i have changed the base to 4.2.x branch |
|
One final thing, could you add a note to the documentation that this setting will impact the path predicate? |
Sorry, I found that there is no webflux related content under the docs of spring-cloud:4.2.x branch, while the main branch has it. Should I cherry-pick from the spring-cloud:main branch and modify it? If so, I plan to add the note to the 'The Path Route Predicate Factory' section in request-predicates-factories.adoc file. |
|
I was just envisioning adding a note to the path predicate documentation that setting |
Do you mean adding the note to the JavaDoc of I also think it would be helpful to include a note in the We can definitely do this after the merge — I’d be happy to follow up on that. |
|
I was thinking specifically here It would be good to have this documented before we merge it so everything is in one PR |
Signed-off-by: 郭付一男 <guofuyinan@gmail.com>
I've updated the documentation as suggested at the specific location you mentioned ( |
Fix Bug : Setting Spring.webflux.base-path All Path Predicates Return 404 Results
Issues :
New Issue : #2944
Three Years Ago Issue : #1759