Skip to content

fix[PPP-6756]: add parsson exclusion to jersey-media-json-processing and explicitly declare 1.1.8 version - #1359

Merged
smmribeiro merged 1 commit into
masterfrom
PPP-6756
Jul 9, 2026
Merged

fix[PPP-6756]: add parsson exclusion to jersey-media-json-processing and explicitly declare 1.1.8 version#1359
smmribeiro merged 1 commit into
masterfrom
PPP-6756

Conversation

@joana-fb

@joana-fb joana-fb commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

@pentaho/tatooine_dev

Copilot AI review requested due to automatic review settings July 9, 2026 15:56
@joana-fb
joana-fb requested a review from a team as a code owner July 9, 2026 15:56

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Pins the Eclipse Parsson JSON-P implementation used by Jersey in the core module by excluding Jersey’s transitive Parsson artifacts and reintroducing them with an explicit, controlled version.

Changes:

  • Added a parsson.version property (1.1.8) to core/pom.xml.
  • Excluded org.eclipse.parsson:parsson and org.eclipse.parsson:parsson-media from jersey-media-json-processing.
  • Added explicit parsson and parsson-media dependencies using the pinned version.

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

@hitachivantarasonarqube

Copy link
Copy Markdown

@buildguy

buildguy commented Jul 9, 2026

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

buildguy commented Jul 9, 2026

Copy link
Copy Markdown
Collaborator

✅ Build finished in 12m 7s

Build command:

mvn clean verify -B -e -Daudit -Djs.no.sandbox -pl core

👌 All tests passed!

Tests run: 440, Failures: 0, Skipped: 1    Test Results


ℹ️ This is an automatic message

@smmribeiro smmribeiro 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.

Looks good!

@smmribeiro
smmribeiro merged commit c0fca83 into master Jul 9, 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.

4 participants