Skip to content

Conversation

@dreis2211
Copy link
Contributor

Hi,

this PR fixes a deprecation in UndertowServletWebServerFactory.

Cheers,
Christoph

P.S.: I'm currently having test failures in NoSuchBeanDefinitionFailureAnalyzerTests that seem unrelated.

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Jul 24, 2018
@snicoll
Copy link
Member

snicoll commented Jul 24, 2018

@dreis2211 that is correct, that's my next task once I am done with the PR I am reviewing.

@snicoll
Copy link
Member

snicoll commented Jul 24, 2018

It's a shame 1.4.26.Final is not available on central.

@dreis2211
Copy link
Contributor Author

Isn't Spring-Boot 2.x on 2.0.11.Final? Or do you mean: it's a shame because it can't be backported to Spring-Boot 1.5.x?

@snicoll
Copy link
Member

snicoll commented Jul 24, 2018

No, it isn't. Only master is on 2.x

@dreis2211
Copy link
Contributor Author

Ah, fair enough. Should we wait then?

@snicoll
Copy link
Member

snicoll commented Jul 24, 2018

I don't know.

@snicoll snicoll added for: team-attention An issue we'd like other members of the team to review and removed for: team-attention An issue we'd like other members of the team to review labels Jul 24, 2018
@snicoll snicoll self-assigned this Jul 25, 2018
@snicoll snicoll added type: task A general task and removed status: waiting-for-triage An issue we've not yet triaged labels Jul 25, 2018
@snicoll snicoll added this to the 2.1.0.M1 milestone Jul 25, 2018
@snicoll snicoll closed this in 2399092 Jul 25, 2018
snicoll added a commit that referenced this pull request Jul 25, 2018
* pr/13873:
  Fix deprecation in UndertowServletWebServerFactory
@snicoll
Copy link
Member

snicoll commented Jul 25, 2018

I've decided to go with this one and I've created a separate issue to remind us to fix the method when we upgrade to 1.4.26.Final.

@dreis2211
Copy link
Contributor Author

Highly appreciated - as always, @snicoll

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