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

JAR checksum of Maven artifact doesn't match GitHub release #6837

Open
der-eismann opened this issue Mar 22, 2024 · 4 comments
Open

JAR checksum of Maven artifact doesn't match GitHub release #6837

der-eismann opened this issue Mar 22, 2024 · 4 comments
Assignees
Labels

Comments

@der-eismann
Copy link

We noticed this while updating our local java environment. Is there a specific reason for this? Previously this was never an issue and paranoid minds would think of manipulation here.

$ curl -O https://repo1.maven.org/maven2/com/datadoghq/dd-java-agent/1.31.2/dd-java-agent-1.31.2.jar
$ sha256sum dd-java-agent-1.31.2.jar
eb4e093aefb2d554b950ecdc9043f198f706b6ca899fce9454e2ee4f8142539d  dd-java-agent-1.31.2.jar
$ curl -LO https://github.com/DataDog/dd-trace-java/releases/download/v1.31.2/dd-java-agent-1.31.2.jar
$ sha256sum dd-java-agent-1.31.2.jar
e640c478cd3cd9c371d1a5ccfe95b77050fc892230e3f4146b3dbb2d7922f807  dd-java-agent-1.31.2.jar
@randomanderson randomanderson self-assigned this Mar 23, 2024
@randomanderson
Copy link
Contributor

This is probably related to #6651 . I believe the sonatype release process itself rebuilds the jar which leads to the difference.

I'll investigate on Monday

@der-eismann
Copy link
Author

Hey @randomanderson, any results from your investigation?

@randomanderson
Copy link
Contributor

@der-eismann That PR highlighted a much bigger issue. Different jobs are rebuilding the jar instead of reusing previously built artifacts. That includes more than just maven vs github release artifacts. For example, kubernetes lib injection images, circleci, etc all have different checksums.

I'm currently working on a larger fix for all of it.

@PerfectSlayer
Copy link
Contributor

This PR #6939 from @randomanderson should address the issue.

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

No branches or pull requests

3 participants