-
Notifications
You must be signed in to change notification settings - Fork 9.1k
HADOOP-18468: jettison 1.5.1 (CVE fix) #4937
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
Conversation
@pjfanning Thank you very much for your contribution, but can the specific fix of CVE be explained in JIRA and PR? I saw the specific description, thank you, no other questions |
@slfan1989 I added a link to https://cve.mitre.org/cgi-bin/cvename.cgi?name=CVE-2022-40149 |
Thank you very much. The pr title can be changed to HADOOP-18468: jettison 1.5.1 (CVE-2022-40149 fix), it will be better. |
container launch problems?
|
I ran this and it was ok:
|
💔 -1 overall
This message was automatically generated. |
do the tests which fail go anywhere near jettison? |
@steveloughran these tests do use jettison so I will need to investigate them. So far, there isn't anything obvious. There are some exceptions in the logs but they are not definitely related to the jettison change. I will continue to look for the cause. |
💔 -1 overall
This message was automatically generated. |
issues with formatting of json in tests
🎊 +1 overall
This message was automatically generated. |
lgtm. +1 pending the checkstyle fixes, which are just line length
|
💔 -1 overall
This message was automatically generated. |
dont see any of those test failures are related, more server launch flakiness. thoughts? |
…#4937) Contributed by PJ Fanning Change-Id: If80704a83bc3bc065be293a89b0c3bb436dcf60f
Contributed by PJ Fanning
…#4937) Contributed by PJ Fanning
Description of PR
A fix for https://cve.mitre.org/cgi-bin/cvename.cgi?name=CVE-2022-40149
How was this patch tested?
For code changes:
LICENSE
,LICENSE-binary
,NOTICE-binary
files?