Skip to content

[UNDERTOW-2736] Align handlers$Builder pattern#1934

Open
baranowb wants to merge 1 commit intoundertow-io:mainfrom
baranowb:UNDERTOW-2736
Open

[UNDERTOW-2736] Align handlers$Builder pattern#1934
baranowb wants to merge 1 commit intoundertow-io:mainfrom
baranowb:UNDERTOW-2736

Conversation

@baranowb
Copy link
Contributor

@baranowb baranowb added enhancement Enhances existing behaviour or code waiting CI check Ready to be merged but waiting for CI check new feature / API change New feature to be introduced or a change to the API (non suitable to minor releases) waiting peer review PRs that edit core classes might require an extra review labels Mar 10, 2026
@baranowb
Copy link
Contributor Author

As is it does not remove old classes, but removes them from services def, as it cause duplicates.

@ropalka ropalka removed the waiting CI check Ready to be merged but waiting for CI check label Mar 10, 2026
@jamezp
Copy link
Member

jamezp commented Mar 11, 2026

Are any of these in a released version of Undertow? If not, just moving the builders. If they're only in main, at least AFAIK, we don't have backwards compatibility to worry about. However, if they're in a released version of Undertow than we do need to deprecate them.

That said, we should add the @Deprecated annotation and we should likely delegate the deprecated build to the non-deprecated builder so fixes only need to happen in one place.

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

Labels

enhancement Enhances existing behaviour or code new feature / API change New feature to be introduced or a change to the API (non suitable to minor releases) waiting peer review PRs that edit core classes might require an extra review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants