Skip to content

Fix multiple CVE #5725

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

Closed
wants to merge 7 commits into from
Closed

Fix multiple CVE #5725

wants to merge 7 commits into from

Conversation

sseide
Copy link
Contributor

@sseide sseide commented Nov 3, 2022

Description

This patch updates some dependencies used to fix multiple security warnings found within these libraries or its dependencies.

Motivation and Context

Additionally i added the new GPG key for the jackson project and remove one old key
that expired in 2016. The other key from jackson project expired just some months ago, so i
let it there. The new key is taken from the Jackson main repositories KEYS file (https://github.com/FasterXML/jackson/blob/master/KEYS)

How Has This Been Tested?

Tested ourself using jmeter with newer libraries and run gradle check

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)

Checklist:

  • My code follows the code style of this project.
  • I have updated the documentation accordingly.

@sseide
Copy link
Contributor Author

sseide commented Dec 22, 2022

i added another update for commons-net to 3.9.0 fixing CVE-2021-37533.
Tests running again without error.

@sseide
Copy link
Contributor Author

sseide commented Jan 4, 2023

and updated xstream to 1.4.20 fixing CVE-2022-40151 and CVE-2022-41966
local tests running again without problem.

@sseide sseide requested a review from FSchumacher January 4, 2023 10:33
@asfgit asfgit closed this in 3e424d7 Jan 4, 2023
@sseide sseide deleted the fix_multiple_cve branch January 4, 2023 15:52
@vlsi vlsi added this to the 5.6 milestone Jun 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants