Skip to content

removing vertx-uri-template as a dependency - #22502

Merged
vmuzikar merged 1 commit into
keycloak:release/22.0from
shawkins:remove-uri-template-backport
Aug 17, 2023
Merged

removing vertx-uri-template as a dependency#22502
vmuzikar merged 1 commit into
keycloak:release/22.0from
shawkins:remove-uri-template-backport

Conversation

@shawkins

Copy link
Copy Markdown
Contributor

backport of #22470 may need to include #22494 (or that can have a separate backport if the issues need to be distinct).

Closes #22468

there's no usage of UriTemplate (smallrye or vertx in keycloak /
fabric8), so it can be removed from server and the operator

Closes keycloak#22468
@shawkins
shawkins requested review from a team as code owners August 16, 2023 17:00
@ghost ghost added the team/cloud-native label Aug 16, 2023
@shawkins

Copy link
Copy Markdown
Contributor Author

The test error is unrelated and captured as #22506

@vmuzikar vmuzikar left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks.

@vmuzikar
vmuzikar enabled auto-merge (squash) August 17, 2023 06:55
@vmuzikar vmuzikar linked an issue Aug 17, 2023 that may be closed by this pull request
@vmuzikar
vmuzikar merged commit 902214f into keycloak:release/22.0 Aug 17, 2023
This was referenced Sep 4, 2023
@stianst stianst mentioned this pull request Nov 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove the vertx-uri-template dependency

2 participants