Skip to content

Conversation

@vtutrinov
Copy link
Contributor

What changes were proposed in this pull request?

ozone-filesystem-hadoop2 transitively depends on net.minidev:json-smart:2.3 that contains a CVE-2021-31684:

https://nvd.nist.gov/vuln/detail/CVE-2021-31684
netplex/json-smart-v2#67

The version of the library needs to be upgraded to 2.5.0

What is the link to the Apache JIRA

https://issues.apache.org/jira/browse/HDDS-10589

How was this patch tested?

Existing hadoop related robot tests

@adoroszlai adoroszlai added the dependencies Pull requests that update a dependency file label Mar 26, 2024
Copy link
Contributor

@adoroszlai adoroszlai left a comment

Choose a reason for hiding this comment

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

As the changes in jar-report.txt indicate, Ozone currently does not depend on json-smart. We should not introduce this dependency.

The dependency on Hadoop 2.x is provided, meaning that users need to supply Hadoop and its dependencies. Therefore, anyone using ozone-filesystem-hadoop2-*.jar can simply upgrade json-smart on their end.

Copy link
Contributor

@dineshchitlangia dineshchitlangia left a comment

Choose a reason for hiding this comment

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

Agree with @adoroszlai

@vtutrinov
Copy link
Contributor Author

As the changes in jar-report.txt indicate, Ozone currently does not depend on json-smart. We should not introduce this dependency.

The dependency on Hadoop 2.x is provided, meaning that users need to supply Hadoop and its dependencies. Therefore, anyone using ozone-filesystem-hadoop2-*.jar can simply upgrade json-smart on their end.

@adoroszlai do you suggest not touch the json-smart dependency in ozone-filesystem-hadoop2 module, but in the place where the module is used? (e.g. in hadoop-ozone/dist/pom.xml -> profile=build-with-ozonefs)

@adoroszlai
Copy link
Contributor

do you suggest not touch the json-smart dependency in ozone-filesystem-hadoop2 module, but in the place where the module is used? (e.g. in hadoop-ozone/dist/pom.xml -> profile=build-with-ozonefs)

I suggest closing this PR, json-smart is not a problem.

@vtutrinov vtutrinov closed this Mar 27, 2024
@vtutrinov
Copy link
Contributor Author

do you suggest not touch the json-smart dependency in ozone-filesystem-hadoop2 module, but in the place where the module is used? (e.g. in hadoop-ozone/dist/pom.xml -> profile=build-with-ozonefs)

I suggest closing this PR, json-smart is not a problem.

Done

@vtutrinov
Copy link
Contributor Author

@adoroszlai @dineshchitlangia thanks for the review

@adoroszlai
Copy link
Contributor

Thanks @vtutrinov for working on these version bumps for security.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants