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

Adds additional backdoored gems. #409

Closed
wants to merge 1 commit into from
Closed

Conversation

jjarmoc
Copy link
Contributor

@jjarmoc jjarmoc commented Aug 29, 2019

This PR adds additional gems affected by CVE-2019-15224 following GHSA's lead by including them under a single CVE and GHSA ID.

I've also included updates for all CVE-2019-* advisories added to GHSA since last sync.

There's a few more (older) GHSA vulns to be sync'd over yet..

Copy link
Member

@reedloden reedloden left a comment

Choose a reason for hiding this comment

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

Thanks so much for the work! There are a few things that need to be fixed before this can be landed, but really do appreciate your efforts so far.

I've commented on a few things only on specific gems, but most of my comments apply to all the entries, so please correct all of them in your next update.

gems/yard/CVE-2019-1020001.yml Outdated Show resolved Hide resolved
gems/simple_captcha2/CVE-2019-14282.yml Outdated Show resolved Hide resolved
gems/ruby-openid/CVE-2019-11027.yml Outdated Show resolved Hide resolved
gems/awesome-bot/CVE-2019-15224.yml Outdated Show resolved Hide resolved
gems/awesome-bot/CVE-2019-15224.yml Outdated Show resolved Hide resolved
gems/awesome-bot/CVE-2019-15224.yml Show resolved Hide resolved
gems/awesome-bot/CVE-2019-15224.yml Outdated Show resolved Hide resolved
gems/awesome-bot/CVE-2019-15224.yml Outdated Show resolved Hide resolved
gems/datagrid/CVE-2019-14281.yml Outdated Show resolved Hide resolved
gems/awesome-bot/CVE-2019-15224.yml Outdated Show resolved Hide resolved
@reedloden
Copy link
Member

Also, not sure why Travis CI passed on a few of these, as there are problems. Should look into that.

@jjarmoc
Copy link
Contributor Author

jjarmoc commented Aug 30, 2019

RE: Travis CI passing them, a handful of the issues (positioning of CVE and ghsa tags in particular) stem from the GHSA sync tool's default. We may want to investigate adjusting the tool's output.

@jjarmoc
Copy link
Contributor Author

jjarmoc commented Sep 2, 2019

Thanks @ingemar, broken links are fixed.

@jjarmoc jjarmoc force-pushed the master branch 2 times, most recently from d943de7 to a5a036d Compare September 2, 2019 17:14
@jjarmoc
Copy link
Contributor Author

jjarmoc commented Sep 5, 2019

Got your comments covered, @reedloden.
I also squashed commits since the history was pretty messy.

phew

Squash commits re: CVE-2019-15224 addition

Fix formatting.

rspec all green now.

First pass at CVE-2019-* GHSA sync

Address @reedloden's PR comments.

Update gems/awesome-bot/CVE-2019-15224.yml

Co-Authored-By: ingemar <ingemar@xox.se>

Update gems/bitcoin_vanity/CVE-2019-15224.yml

Co-Authored-By: ingemar <ingemar@xox.se>

Fix incorrect links

Update titles for CVE-2019-15224 related entries.

Update description

Remove redundant reference.

Fix GHSA formatting

fix GHSA formatting

List versions consistently.

Fix GHSA formatting

Add better reference

Multiple changes:

- Update reference URL
- Update description
- Move CVE and CVSS entries

Multiple Changes:

- Moved CVE & CVSS entires
- Changed reference URL
- Fixed version formatting
- Reformatted description

Update reference URL

Add GHSA, fix description.

Spaces > Tabs

Fight me.

Spaces > Tabs

Fight me.

Spaces > Tabs

Change my mind.
@reedloden
Copy link
Member

I downloaded this PR locally, made some edits, and just merged this to master. It didn't close the PR for some reason, but this has been landed!

7a69f6d

Thank you so much, @jjarmoc, for all the hard work on this. Really do appreciate it! This project would not survive without all the volunteer contributors such as yourself.

@reedloden reedloden closed this Sep 6, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants