Skip to content

Pin Maven Dependencies to SHAs via Lockfile#739

Merged
hectorcast-db merged 1 commit intomainfrom
pin-maven-dependencies
Mar 27, 2026
Merged

Pin Maven Dependencies to SHAs via Lockfile#739
hectorcast-db merged 1 commit intomainfrom
pin-maven-dependencies

Conversation

@rauchy
Copy link
Contributor

@rauchy rauchy commented Mar 26, 2026

This PR adds SHA-256 hash verification for all Maven dependencies using the
maven-lockfile plugin.
This is the Maven equivalent of Go's go.sum or pip's --require-hashes.

Maven's default checksum policy is permissive (warns on mismatch but doesn't
fail). The lockfile plugin generates a committed lockfile.json with SHA-256
hashes for every dependency (direct + transitive) and validates them in CI.

Changes:

  • Added maven-lockfile plugin (v5.5.2) to root pom.xml
  • Generated lockfile.json for both parent and SDK modules
  • Added check-lock job to push.yml that validates on every PR
  • Added make lock / make check-lock targets

To update after dependency changes: run make lock and commit the updated
lockfile(s).

@rauchy rauchy requested a review from parthban-db March 26, 2026 13:22
{
"artifactId": "databricks-sdk-parent",
"groupId": "com.databricks",
"version": "0.103.0",
Copy link
Contributor

Choose a reason for hiding this comment

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

We need to upgrade the tagging or some script to update this; otherwise, we can forget to update this.

{
"artifactId": "databricks-sdk-java",
"groupId": "com.databricks",
"version": "0.103.0",
Copy link
Contributor

Choose a reason for hiding this comment

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

ditto

uses: actions/checkout@ee0669bd1cc54295c223e0bb666b733df41de1c5 # v2.7.0

- name: Validate lockfile
run: make check-lock
Copy link
Contributor

Choose a reason for hiding this comment

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

We should run this prior to the release as well.

@hectorcast-db hectorcast-db merged commit e985d80 into main Mar 27, 2026
@hectorcast-db hectorcast-db deleted the pin-maven-dependencies branch March 27, 2026 14:45
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 this pull request may close these issues.

4 participants