-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Jackson upgrade for vulnerability issue #405
base: master
Are you sure you want to change the base?
Conversation
@fmbenhassine do you have sometime to look at this and maybe merge it? |
Yes. I am planning to do a release soon. I will make sure to include updated dependencies. |
Any update on this? |
@@ -36,7 +36,7 @@ | |||
<mockito.version>3.6.0</mockito.version> | |||
<system-lambda.version>1.1.1</system-lambda.version> | |||
<slf4j.version>1.7.30</slf4j.version> | |||
<jackson.version>2.11.3</jackson.version> | |||
<jackson.version>2.14.0</jackson.version> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
<jackson.version>2.14.0</jackson.version> | |
<jackson.version>2.15.2</jackson.version> |
@junaidwarsivd Thank you very much. |
@fmbenhassine Any update on the release? Also possible to look into this issue where exception is happening in JDK 21. Thanks a lot! |
Why not merge? |
current version of Jackson being used in release easyrules release (4.1.0) has a vulnerability issues
Deserialization of Untrusted Data (High) - CWE-502
XML External Entity (XXE) Injection (High) - CWE-611 - CVE-2020-25649
Denial of Service (DoS) - CWE-400
this PR is for the upgrade for jackson databind dependency which covers the issues mentioned above