Support setting Spring.webflux.base-path and supporting it in path predicate#2984
Support setting Spring.webflux.base-path and supporting it in path predicate#2984ryanjbaxter merged 2 commits intospring-cloud:4.2.xfrom
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! |
I will add a test ASAP |
920d2ab to
1ceaee8
Compare
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? |
|
Can you submit this against the 4.2.x branch? |
…edicate Signed-off-by: 郭付一男 <[email protected]>
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: 郭付一男 <[email protected]>
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