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

Fix dependencies for Grizzly connector #4917

Closed
zUniQueX opened this issue Nov 28, 2021 · 3 comments · Fixed by #4921
Closed

Fix dependencies for Grizzly connector #4917

zUniQueX opened this issue Nov 28, 2021 · 3 comments · Fixed by #4921

Comments

@zUniQueX
Copy link
Contributor

zUniQueX commented Nov 28, 2021

The grizzly-connector declares dependencies for grizzly-websockets and connection-pool, while grizzly-http-client already has dependencies on those. This is causing conflicts at the dependency convergence in the maven enforcer plugin in projects using this connector since grizzly-http-client uses the dependency version 2.4.3, while the pom of Jersey defines the version as 2.4.4.

Would it be possible to either remove these dependencies or to add exclusions for them to the dependency declaration of grizzly-http-client? Else one cannot use this connector when having dependency convergence enabled.

zUniQueX added a commit to zUniQueX/jersey that referenced this issue Dec 2, 2021
Exclude declared dependencies with other versions from grizzly-http-client

Signed-off-by: Steffen Nießing <zuniquex@protonmail.com>
@jansupol jansupol linked a pull request Dec 3, 2021 that will close this issue
jansupol pushed a commit that referenced this issue Dec 6, 2021
Exclude declared dependencies with other versions from grizzly-http-client

Signed-off-by: Steffen Nießing <zuniquex@protonmail.com>
@zUniQueX
Copy link
Contributor Author

@jansupol As the next 3.0.x version is released, are there any release plans for 2.36 to get this fixed?

@jansupol
Copy link
Contributor

Currently, only a dot release can be made, until a release review (which needs to be done once a year). We will go through the release review and we plan the release for March.

@zUniQueX
Copy link
Contributor Author

Okay, thanks for clarifying this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants