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

Jackson upgrade for vulnerability issue #405

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

junaidwarsivd
Copy link

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

@SebaMutuku
Copy link

@fmbenhassine do you have sometime to look at this and maybe merge it?

@fmbenhassine
Copy link
Member

Yes. I am planning to do a release soon. I will make sure to include updated dependencies.

@melloware
Copy link

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>

Choose a reason for hiding this comment

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

Suggested change
<jackson.version>2.14.0</jackson.version>
<jackson.version>2.15.2</jackson.version>

@pdob-git
Copy link

@junaidwarsivd Thank you very much.
I have updated my project from your fork 😄 👍

@Joe2k
Copy link

Joe2k commented Feb 8, 2024

@fmbenhassine Any update on the release? Also possible to look into this issue where exception is happening in JDK 21. Thanks a lot!

@xiangdyzz
Copy link

Why not merge?

@xiangdyzz
Copy link

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

Successfully merging this pull request may close these issues.

8 participants