Skip to content

up rexml to a version that doesnt include the ddos vulnerability#806

Closed
jeremiahlukus wants to merge 1 commit intogettalong:masterfrom
jeremiahlukus:master
Closed

up rexml to a version that doesnt include the ddos vulnerability#806
jeremiahlukus wants to merge 1 commit intogettalong:masterfrom
jeremiahlukus:master

Conversation

@jeremiahlukus
Copy link

REXML contains a denial of service vulnerability (CVE-2024-35176)

Impact The REXML gem before 3.2.6 has a DoS vulnerability when it parses an XML that has many <s in an attribute value. If you need to parse untrusted XMLs, you many be impacted to this vulnerability. ### Patches The REXML gem 3.2.7 or later include the patch to fix this vulnerability. ### Workarounds Don't parse untrusted XMLs. ### References * https://www.ruby-lang.org/en/news/2024/05/16/dos-rexml-cve-2024-35176/

^ pasted from NewRelic

@monicao
Copy link

monicao commented Aug 26, 2024

It looks like more vulnerabilities were found since May.

Consider bumping the version of rexml to '>= 3.3.6', because this CVE-2024-43398 reports the DoS vulnerability is there in prior versions.

@jeremiahlukus
Copy link
Author

I would if it would get merged since this didnt get merged its unlikely another version bump will.

@etherz10
Copy link

@gettalong Any chance of bumping the minimum rexml version required to 3.3.6 and then merging this?

@gettalong gettalong self-assigned this Nov 16, 2024
@gettalong
Copy link
Owner

I have changed the version to >= 3.3.6 and will do a release within a week.

@gettalong gettalong closed this Nov 16, 2024
@etherz10
Copy link

@gettalong I see you've just made a 2.5.0 release that requires 3.3.6 but theres a CVE thats been fixed in 3.3.9 -- Could you do a .1 release and bump that dep again please?

https://nvd.nist.gov/vuln/detail/CVE-2024-49761

@gettalong
Copy link
Owner

@etherz10 I have just pushed kramdown 2.5.1 with the necessary change.

@jeremiahlukus
Copy link
Author

Thanks man!

@etherz10
Copy link

champion, thank you

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.

4 participants