Skip to content

fix[SP-7345]: Add explicit dependency to bcprov-jdk15to18#1876

Merged
befc merged 2 commits into
10.2from
SP-7345_2
May 26, 2026
Merged

fix[SP-7345]: Add explicit dependency to bcprov-jdk15to18#1876
befc merged 2 commits into
10.2from
SP-7345_2

Conversation

@befc

@befc befc commented May 26, 2026

Copy link
Copy Markdown
Contributor

This fixes emr770 hadoop shim after the exclusion made in the previous PR: #1874

Copilot AI review requested due to automatic review settings May 26, 2026 10:38
@befc befc requested a review from a team as a code owner May 26, 2026 10:38

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 aims to restore the EMR 7.7.0 shim’s effective Bouncy Castle provider after bcprov-jdk15on was excluded transitively in PR #1874, by ensuring bcprov-jdk15to18 is explicitly present on the driver’s dependency list.

Changes:

  • Added an explicit org.bouncycastle:bcprov-jdk15to18 dependency near the hadoop-common dependency block.

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

Comment thread shims/emr770/driver/pom.xml Outdated
@buildguy

Copy link
Copy Markdown
Collaborator

✅ Build finished in 19m 33s

Build command:

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

❗ No tests found!

ℹ️ This is an automatic message

@hitachivantarasonarqube

Copy link
Copy Markdown

@befc befc merged commit 3ba572e into 10.2 May 26, 2026
1 of 2 checks passed
@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.


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