Skip to content

Conversation

@dreis2211
Copy link
Contributor

Hi,

this PR fixes a compilation error caused by a missing implementation in AbstractWebFluxEndpointHandlerMapping of Set<PathPattern> getMappingPathPatterns(RequestMappingInfo mapping) that seems to have been introduced recently.

Cheers,
Christoph

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Apr 6, 2019
@dreis2211
Copy link
Contributor Author

Test failure could be related, but I have the weird feeling this might be related to #16410 .

@snicoll
Copy link
Member

snicoll commented Apr 6, 2019

Me too even though we had green builds after this was merged. Nothing jumps at me that might have changed in framework in the meantime.

Paging @sdeleuze.

@snicoll snicoll self-assigned this Apr 6, 2019
@snicoll snicoll added type: task A general task and removed status: waiting-for-triage An issue we've not yet triaged labels Apr 6, 2019
@snicoll snicoll added this to the 2.2.0.M2 milestone Apr 6, 2019
@snicoll snicoll closed this in 52cd9c6 Apr 6, 2019
snicoll added a commit that referenced this pull request Apr 6, 2019
* pr/16476:
  Fix compilation error
@snicoll
Copy link
Member

snicoll commented Apr 6, 2019

@dreis2211 I might have fixed it in 84735f5 - We've harmonized the context hierarchy recently and those tests were calling refresh twice (which is not something the regular hierarchy allows). Arguably, that was a problem in the test in the first place so I've moved them to use the ApplicationContextRunner utility.

@dreis2211
Copy link
Contributor Author

@snicoll Thanks for the heads up.

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

Labels

type: task A general task

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants