Skip to content

Commit c6ab4de

Browse files
authored
Skip OWASP dependency check in release build step (#1286)
## Summary - Skip OWASP dependency check in the "Build dependencies" step of `release.yml` and `release-thin.yml` by adding `-Ddependency-check.skip=true` - The deploy step already runs the OWASP check with the NVD API key — running it in the build step was redundant and caused the v3.3.1 release to fail (downloaded 338K NVD records without an API key, then hit a false positive on CVE-2026-25087) ## Test plan - [ ] After merging, delete the v3.3.1 release and tag, then re-create from GitHub UI to re-trigger the release workflow - [ ] Verify the release workflow completes successfully NO_CHANGELOG=true OVERRIDE_FREEZE=true This pull request was AI-assisted by Isaac. Signed-off-by: Madhavendra Rathore <madhavendra.rathore@databricks.com>
1 parent aa29042 commit c6ab4de

2 files changed

Lines changed: 4 additions & 2 deletions

File tree

.github/workflows/release-thin.yml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,8 @@ jobs:
5050
# GPG signing is skipped since we're only installing locally, not publishing.
5151
- name: Build dependencies
5252
run: |
53-
mvn -Prelease clean install --batch-mode -pl jdbc-core -am -Dgpg.skip=true
53+
mvn -Prelease clean install --batch-mode -pl jdbc-core -am -Dgpg.skip=true \
54+
-Ddependency-check.skip=true
5455
5556
# Step 2: Deploy only the thin JAR module to Maven Central
5657
# We don't use -am here to avoid the central-publishing-maven-plugin

.github/workflows/release.yml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,8 @@ jobs:
3535
# GPG signing is skipped since we're only installing locally, not publishing.
3636
- name: Build dependencies
3737
run: |
38-
mvn -Prelease clean install --batch-mode -pl jdbc-core -am -Dgpg.skip=true
38+
mvn -Prelease clean install --batch-mode -pl jdbc-core -am -Dgpg.skip=true \
39+
-Ddependency-check.skip=true
3940
4041
# Step 2: Deploy only the uber JAR module to Maven Central
4142
# We don't use -am here to avoid the central-publishing-maven-plugin

0 commit comments

Comments
 (0)