Skip to content

update json-smart version#612

Merged
siddhijain merged 2 commits intodevfrom
SJAIN/update-json-dependency
Mar 27, 2023
Merged

update json-smart version#612
siddhijain merged 2 commits intodevfrom
SJAIN/update-json-dependency

Conversation

@siddhijain
Copy link
Contributor

@siddhijain siddhijain commented Mar 24, 2023

Updated version due to a security vulnerability in 2.4.8.

@siddhijain siddhijain requested a review from bgavrilMS March 24, 2023 20:39
@jeanbisutti
Copy link

Hi @bgavrilMS
I work on Java Application Insights. MSAL is a transitive dependency.
One of our users would like to know if CVE-2023-1370 is exploitable. Would MSAL use json-smart features potentially exposed to CVE-2023-1370? Thanks.

Updated json-smart version to a 'bug-free' version
@siddhijain
Copy link
Contributor Author

Hi @bgavrilMS I work on Java Application Insights. MSAL is a transitive dependency. One of our users would like to know if CVE-2023-1370 is exploitable. Would MSAL use json-smart features potentially exposed to CVE-2023-1370? Thanks.

@jeanbisutti Msal Java uses a class of net-minidev.json-smart to pass json objects to another library 'nimbus' for processing. From the way msal java uses json-smart, it does not look like it will be susceptible to the vulnerability. However, to be on the safer side, this PR updates the json-smart version to a secured version 2.4.10. Once merged, I will release a hotfix that you can consume in your application.   

@jeanbisutti
Copy link

Thanks a lot @siddhijain!

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