Skip to content

Conversation

@michaelkedar
Copy link
Member

We didn't catch this when merging:

  • link was 404-ing, since it's expecting the prefix of the path to the OSV record
  • human_link was using CVE_ID instead of BUG_ID

Copy link
Contributor

@another-rex another-rex left a comment

Choose a reason for hiding this comment

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

Nice!

@another-rex another-rex enabled auto-merge (squash) October 29, 2025 04:52
@another-rex another-rex merged commit d7a1d2b into google:master Oct 29, 2025
18 checks passed
jess-lowe pushed a commit that referenced this pull request Oct 31, 2025
Follow up to #4192 & #4267

The link only existed with the CVE ID in it and not the OSV ID. I
created a redirect at the given path.

I've verified a few test cases manually, but still can't see any info on
`https://api.test.osv.dev/v1experimental/importfindings/eef`.
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.

2 participants