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

Dependency convergence issue not fixed #3810

Closed
Venorcis opened this issue Mar 6, 2023 · 4 comments
Closed

Dependency convergence issue not fixed #3810

Venorcis opened this issue Mar 6, 2023 · 4 comments
Labels
feature-request A feature should be added or improved. p2 This is a standard priority issue

Comments

@Venorcis
Copy link

Venorcis commented Mar 6, 2023

Describe the bug

The issue as originally described in #3787 was unfortunately not fixed in release 2.20.14 (or later).

Expected Behavior

Dependency convergence error for commons-codec:commons-codec to be resolved.

Current Behavior

Dependency convergence error for commons-codec:commons-codec remains present.

Reproduction Steps

Add a dependency on software.amazon.awssdk:s3:jar:2.20.17 and do a mvn install including the maven-enforcer-plugin with <dependencyConvergence /> set.

Possible Solution

No response

Additional Information/Context

Latest error output:

Dependency convergence error for commons-codec:commons-codec:jar:1.11 paths to dependency are:
+-<my-project>:jar:2.0.0-SNAPSHOT
  +-software.amazon.awssdk:s3:jar:2.20.17:compile
    +-software.amazon.awssdk:apache-client:jar:2.20.17:runtime
      +-org.apache.httpcomponents:httpclient:jar:4.5.14:runtime
        +-commons-codec:commons-codec:jar:1.11:runtime
and
+-<my-project>:jar:2.0.0-SNAPSHOT
  +-software.amazon.awssdk:s3:jar:2.20.17:compile
    +-software.amazon.awssdk:apache-client:jar:2.20.17:runtime
      +-commons-codec:commons-codec:jar:1.15:runtime

AWS Java SDK version used

2.20.17

JDK version used

openjdk 17.0.6

Operating System and version

ubuntu 22.04.2

@Venorcis Venorcis added bug This issue is a bug. needs-triage This issue or PR still needs to be triaged. labels Mar 6, 2023
@debora-ito debora-ito added needs-review This issue or PR needs review from the team. and removed needs-triage This issue or PR still needs to be triaged. labels Mar 7, 2023
@debora-ito
Copy link
Member

debora-ito commented Mar 14, 2023

We'll take look on how to improve our dependency convergence experience. Currently we don't use the enforcer plugin in our build process, but we'll consider adding it.

Changing to feature request, as this is an improvement more than a bug.

@debora-ito debora-ito added feature-request A feature should be added or improved. p2 This is a standard priority issue and removed bug This issue is a bug. needs-review This issue or PR needs review from the team. labels Mar 14, 2023
@Venorcis
Copy link
Author

We'll take look on how to improve our dependency convergence experience. Currently we don't use the enforcer plugin in our build process, but we'll consider adding it.

Thank you!

Changing to feature request, as this is an improvement more than a bug.

As my organization does enforce this, for us it is a blocker for updating beyond 2.20.7.

@Venorcis
Copy link
Author

Version 2.20.44 seems to work again (i.e. without dependency convergence issues); from the changelog I cannot really see which version fixed this (I didn't try every single one), but I'll close this issue in any case 😄

@github-actions
Copy link

⚠️COMMENT VISIBILITY WARNING⚠️

Comments on closed issues are hard for our team to see.
If you need more assistance, please open a new issue that references this one.
If you wish to keep having a conversation with other community members under this issue feel free to do so.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature-request A feature should be added or improved. p2 This is a standard priority issue
Projects
None yet
Development

No branches or pull requests

2 participants