Skip to content

Update vulnerable libraries#1756

Open
dterrybd wants to merge 15 commits into
masterfrom
dev/dterry/IDETECT-5080-update-vuln-libraries
Open

Update vulnerable libraries#1756
dterrybd wants to merge 15 commits into
masterfrom
dev/dterry/IDETECT-5080-update-vuln-libraries

Conversation

@dterrybd
Copy link
Copy Markdown
Contributor

This locks json-path to a version without vulnerabilities when used by other things like Spring.

It also moves to newer jackson and blackduck-common library versions that have fixes we should bring into Detect.

@dterrybd dterrybd self-assigned this May 12, 2026
Comment thread build.gradle Outdated
javaSourceCompatibility = 8
}
ext['logback.version'] = '1.2.13'
ext['json-path.version'] = '2.9.0'
Copy link
Copy Markdown
Collaborator

@zahidblackduck zahidblackduck May 13, 2026

Choose a reason for hiding this comment

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

json-path:2.9.0 possibly brings json-smart:2.5.0 which is an exploitable dependency as per the pop scan in detect-scripts as well as the detect-docker alpine image. This has been seen in upstream libraries.
So, we patched the upstream integration-common from json-path:2.9.0 to json-path:2.10.0 which is now bringing the safer json-smart:2.6.0.
But, as we are pinning this here, it might bring the json-smart:2.5.0 again which is exploitable.
Could you consider pinning json-path to 2.10.0 if that doesn't have any breaking change?

json-path:2.10.0 changelog for reference: https://github.com/json-path/JsonPath/releases/tag/json-path-2.10.0

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Thank you for the comment and details. Updated to pin to the version you suggested. Confirmed 2.10 of json-path comes and 2.6 of json-smart.

@dterrybd dterrybd requested a review from zahidblackduck May 14, 2026 00:19
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.

2 participants