Add CRAN+Bioconductor ecosystems #176
Conversation
|
I think it's okay to mark this as |
Co-authored-by: Tyler Finethy <tylfin@gmail.com> Signed-off-by: Michael Chirico <michaelchirico4@gmail.com>
docs/schema.md
Outdated
| <ul> | ||
| <li>How to contribute: <a href="https://github.com/RConsortium/r-advisory-database#readme">https://github.com/RConsortium/r-advisory-database#readme</a></li> | ||
| <li>Source URL: <code>https://osv.dev/vulnerability/<ID></code></li> | ||
| <li>OSV Formatted URL: <code>https://github.com/RConsoritum/r-advisory-database/<package>/<ID>.yaml</code> (unofficial)</li> |
There was a problem hiding this comment.
There is a typo in the URL
| <li>OSV Formatted URL: <code>https://github.com/RConsoritum/r-advisory-database/<package>/<ID>.yaml</code> (unofficial)</li> | |
| <li>OSV Formatted URL: <code>https://github.com/RConsortium/r-advisory-database/<package>/<ID>.yaml</code> (unofficial)</li> |
There was a problem hiding this comment.
Also, does the constructed URL work though? E.g.
https://github.com/RConsortium/r-advisory-database/readxl/RSEC-2023-0.yaml
would not lead to the advisory if I'm not mistaken.
There was a problem hiding this comment.
This should work directly:
| <li>OSV Formatted URL: <code>https://github.com/RConsoritum/r-advisory-database/<package>/<ID>.yaml</code> (unofficial)</li> | |
| <li>OSV Formatted URL: <code>https://raw.githubusercontent.com/RConsortium/r-advisory-database/main/vulns/<package>/<ID>.yaml</code></li> |
For example https://raw.githubusercontent.com/RConsortium/r-advisory-database/main/vulns/readxl/RSEC-2023-0.yaml
There was a problem hiding this comment.
I've fixed the link here.
Signed-off-by: Oliver Chang <oliverchang@users.noreply.github.com>
| <ul> | ||
| <li>How to contribute: <a href="https://github.com/RConsortium/r-advisory-database#readme">https://github.com/RConsortium/r-advisory-database#readme</a></li> | ||
| <li>Source URL: <code>https://osv.dev/vulnerability/<ID></code></li> | ||
| <li>OSV Formatted URL: <code>https://github.com/RConsortium/r-advisory-database/blob/main/<package>/<ID>.yaml</code> (unofficial)</li> |
There was a problem hiding this comment.
@oliverchang I think this is still wrong, missing /vulns in the path :D
There was a problem hiding this comment.
Doh, thanks for catching.
Can you please raise a PR to fix this?
There was a problem hiding this comment.
Doh, thanks for catching.
Can you please raise a PR to fix this?
There was a problem hiding this comment.
Sure, I'll push it when I get a minute.
Closes #175.
I didn't quite understand "Source URL" vs. "OSV Formatted URL" so I took a guess.
cc @tylfin as upstream maintainer. I marked this as "unofficial" despite R Consortium backing, please CMIIW.