Skip to content

Conversation

@kiskoza
Copy link

@kiskoza kiskoza commented Jan 6, 2026

@jasnow
Copy link
Contributor

jasnow commented Jan 6, 2026

Three references that can have cvss v2 to v4 are
nvd.nist.gov, https://github.com/advisories, and the security advisory inside the repo.

@jasnow
Copy link
Contributor

jasnow commented Jan 6, 2026

Two things you can do to hep @postmodern before pushing a PR is:

This probably needs to be added to README.

@kiskoza
Copy link
Author

kiskoza commented Jan 6, 2026

Thanks for the feedback, I hope it looks better now

@jasnow
Copy link
Contributor

jasnow commented Jan 6, 2026

Add "cvss_v4: 8.8" based on nvd.nist.gov reference.

Copy link
Member

@postmodern postmodern left a comment

Choose a reason for hiding this comment

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

YAML formatting of description: is wrong. Should use inline block text for readability.

url: https://github.com/jnunemaker/httparty/security/advisories/GHSA-hm5p-x4rq-38w4
title: httparty Has Potential SSRF Vulnerability That Leads to API Key Leakage
date: 2025-12-23
description: "## Summary\n\nThere may be an SSRF vulnerability in httparty. This issue
Copy link
Member

Choose a reason for hiding this comment

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

description: should use inline block text:

description: |
  Foo bar baz ...
  quux ...

You could copy/paste the text into irb to remove the YAML String encoding:

$ irb
>> puts("giant string here ...")

and then copy/paste the output back into the YAML.

Copy link
Member

@postmodern postmodern Jan 8, 2026

Choose a reason for hiding this comment

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

If you paste the full YAML advisory into this command it will reformat the description properly:

ruby -r yaml -e 'puts YAML.load(STDIN.read)["description"]'

Then indent each line by two spaces and copy/paste back into the advisory under description: |.

Copy link
Author

Choose a reason for hiding this comment

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

thanks, I updated it

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