Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

TEZ-4424: [CVE-2021-3918] Upgrade json-schema from 0.2.3 to 0.4.0 #222

Merged
merged 1 commit into from
Jun 20, 2022

Conversation

amanraj2520
Copy link
Contributor

[TEZ-4424] [CVE-2021-3918] Upgrade json-schema from 0.2.3 to 0.4.0 to fix the vulnerability.
Link to JIRA : https://issues.apache.org/jira/browse/TEZ-4424

Link to parent JIRA : https://issues.apache.org/jira/browse/TEZ-4419

RFC documentation : https://github.com/yarnpkg/rfcs/blob/master/implemented/0000-selective-versions-resolutions.md

@amanraj2520
Copy link
Contributor Author

@guptanikhil007 Can you please approve this PR once the build pipeline is complete?

@tez-yetus
Copy link

🎊 +1 overall

Vote Subsystem Runtime Comment
+0 🆗 reexec 15m 3s Docker mode activated.
_ Prechecks _
+1 💚 dupname 0m 0s No case conflicting files found.
+1 💚 @author 0m 0s The patch does not contain any @author tags.
_ master Compile Tests _
_ Patch Compile Tests _
+1 💚 whitespace 0m 0s The patch has no whitespace issues.
_ Other Tests _
+1 💚 asflicense 1m 23s The patch does not generate ASF License warnings.
17m 20s
Subsystem Report/Notes
Docker ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-222/1/artifact/out/Dockerfile
GITHUB PR #222
Optional Tests dupname asflicense
uname Linux 9e5fe9b1d5e8 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux
Build tool maven
Personality personality/tez.sh
git revision master / c6eb047
Max. process+thread count 51 (vs. ulimit of 5500)
modules C: tez-ui U: tez-ui
Console output https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-222/1/console
versions git=2.25.1 maven=3.6.3
Powered by Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

Copy link
Contributor

@guptanikhil007 guptanikhil007 left a comment

Choose a reason for hiding this comment

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

Can you verify if the tests are running successfully?

@amanraj2520
Copy link
Contributor Author

Yes they are the build is green
Also tested in my local, the phantom JS tests are running fine

@amanraj2520
Copy link
Contributor Author

Also the TravisCI build has passed. LGTM

@amanraj2520
Copy link
Contributor Author

@abstractdog Can you please approve and merge these changes

@amanraj2520
Copy link
Contributor Author

@abstractdog Just a gentle reminder, can you please approve these changes and merge them?

@amanraj2520
Copy link
Contributor Author

@abstractdog Can you please approve and merge these changes

@abstractdog abstractdog self-requested a review June 20, 2022 06:44
@abstractdog abstractdog merged commit 8ffa32d into apache:master Jun 20, 2022
@abstractdog
Copy link
Contributor

LGTM, checked locally, merged to master

@amanraj2520
Copy link
Contributor Author

@abstractdog Thanks a lot.

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