Skip to content

build[SP-7278]: Remove bouncycastle.version property to force inherit from maven-parent-pom#1865

Merged
befc merged 1 commit into
11.0from
SP-7278_2
May 22, 2026
Merged

build[SP-7278]: Remove bouncycastle.version property to force inherit from maven-parent-pom#1865
befc merged 1 commit into
11.0from
SP-7278_2

Conversation

@befc

@befc befc commented May 21, 2026

Copy link
Copy Markdown
Contributor

⚠️ this also needs #1864, that had removed a duplicate of bouncycastle.version in same pom.xml file.

Copilot AI review requested due to automatic review settings May 21, 2026 17:36
@befc befc requested a review from a team as a code owner May 21, 2026 17:36

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR targets build configuration in the Apache shim driver by adjusting how the bouncycastle.version Maven property is defined, with the stated goal (per title/description) of inheriting the version from the parent POM.

Changes:

  • Removes a duplicate <bouncycastle.version> property entry from shims/apache/driver/pom.xml.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines 30 to 32
<zookeeper.version>3.9.3</zookeeper.version>
<bouncycastle.version>1.78</bouncycastle.version>
<curator.version>5.6.0</curator.version>
@hitachivantarasonarqube

Copy link
Copy Markdown

@buildguy

Copy link
Copy Markdown
Collaborator

👍 Frogbot scanned this pull request and did not find any new security issues.

Note:

Frogbot also supports Contextual Analysis, Secret Detection, IaC and SAST Vulnerabilities Scanning. This features are included as part of the JFrog Advanced Security package, which isn't enabled on your system.


@buildguy

Copy link
Copy Markdown
Collaborator

✅ Build finished in 10m 15s

Build command:

mvn clean verify -B -e -Daudit -Djs.no.sandbox -pl shims/apache/driver

❗ No tests found!

ℹ️ This is an automatic message

@befc befc merged commit 055adaa into 11.0 May 22, 2026
3 checks passed
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.

5 participants