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.

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

Support setting Spring.webflux.base-path and supporting it in path predicate #2984

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

Guofuyinan
Copy link

Fix Bug : Setting Spring.webflux.base-path All Path Predicates Return 404 Results

Issues :
New Issue : #2944
Three Years Ago Issue : #1759

@pivotal-cla
Copy link

@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.

@pivotal-cla
Copy link

@Guofuyinan Thank you for signing the Contributor License Agreement!

Copy link
Member

@spencergibb spencergibb left a 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

@Guofuyinan
Copy link
Author

Guofuyinan commented Jun 20, 2023

please add a test

I will add a test ASAP

…ath Predicates Return 404 Results

Changing the order of imports

Revert code style
@spencergibb
Copy link
Member

Looks like some checkstyle issues.

@Guofuyinan
Copy link
Author

Looks like some checkstyle issues.

Try again

@scott-avery
Copy link

@spencergibb any update?

@spencergibb spencergibb changed the title Fix Bug : Setting Spring.webflux.base-path All Path Predicates Return 404 Results Support setting Spring.webflux.base-path and supporting it in path predicate Mar 11, 2024
@spencergibb
Copy link
Member

We will get this included in the 2024.0 release train.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Todo
Development

Successfully merging this pull request may close these issues.

None yet

5 participants