feat(vulnerabilities): add reference_link field with priority-based URL selection#3988
feat(vulnerabilities): add reference_link field with priority-based URL selection#3988
reference_link field with priority-based URL selection#3988Conversation
…RL selection Signed-off-by: anilb <epipav@gmail.com>
|
Your PR title doesn't contain a Jira issue key. Consider adding it for better traceability. Example:
Projects:
Please add a Jira issue key to your PR title. |
|
|
|
Your PR title doesn't contain a Jira issue key. Consider adding it for better traceability. Example:
Projects:
Please add a Jira issue key to your PR title. |
|
Your PR title doesn't contain a Jira issue key. Consider adding it for better traceability. Example:
Projects:
Please add a Jira issue key to your PR title. |
|
Your PR title doesn't contain a Jira issue key. Consider adding it for better traceability. Example:
Projects:
Please add a Jira issue key to your PR title. |
...es/apps/git_integration/src/crowdgit/services/vulnerability_scanner/vulnerability_scanner.go
Show resolved
Hide resolved
services/apps/git_integration/src/crowdgit/services/vulnerability_scanner/db.go
Show resolved
Hide resolved
There was a problem hiding this comment.
Pull request overview
Adds a reference_link field to vulnerability records end-to-end (scanner → DB upsert → Tinybird datasource → Tinybird list pipe) so clients can display a canonical URL for each vulnerability.
Changes:
- Extend vulnerability scanner output to compute and persist a single “best” reference URL per vulnerability.
- Persist
reference_linkin the DB upsert and expose it in Tinybird ingestion schema. - Return
referenceLinkfrom the Tinybirdvulnerabilities_listpipe output.
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| services/apps/git_integration/src/crowdgit/services/vulnerability_scanner/vulnerability_scanner.go | Adds best-reference URL selection and includes it in normalized vulnerability output. |
| services/apps/git_integration/src/crowdgit/services/vulnerability_scanner/types.go | Adds ReferenceLink to the Vulnerability struct JSON payload. |
| services/apps/git_integration/src/crowdgit/services/vulnerability_scanner/db.go | Writes reference_link into the vulnerabilities upsert (insert + conflict update). |
| services/libs/tinybird/datasources/vulnerabilities.datasource | Adds referenceLink column mapping from record.reference_link. |
| services/libs/tinybird/pipes/vulnerabilities_list.pipe | Returns referenceLink in the aggregated vulnerabilities list response. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
...es/apps/git_integration/src/crowdgit/services/vulnerability_scanner/vulnerability_scanner.go
Show resolved
Hide resolved
...es/apps/git_integration/src/crowdgit/services/vulnerability_scanner/vulnerability_scanner.go
Outdated
Show resolved
Hide resolved
Signed-off-by: anilb <epipav@gmail.com>
…dDotDev/crowd.dev into feat/vulnerabilities-reference-link
|
Your PR title doesn't contain a Jira issue key. Consider adding it for better traceability. Example:
Projects:
Please add a Jira issue key to your PR title. |
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
There are 2 total unresolved issues (including 1 from previous review).
Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.
…bird Signed-off-by: anilb <epipav@gmail.com>
|
Your PR title doesn't contain a Jira issue key. Consider adding it for better traceability. Example:
Projects:
Please add a Jira issue key to your PR title. |
|
Your PR title doesn't contain a Jira issue key. Consider adding it for better traceability. Example:
Projects:
Please add a Jira issue key to your PR title. |

Note
Medium Risk
Touches persistence/query surfaces (Postgres upsert + Tinybird schema/pipe), so deployments must ensure the underlying
vulnerabilities.reference_linkcolumn exists and data shape changes won’t break downstream consumers.Overview
Adds a new
reference_linkfield to vulnerability findings end-to-end: the scanner now derives a best reference URL (preferring an NVD link when a CVE exists, otherwise prioritizing OSV reference types) and persists it during the vulnerabilities upsert.Updates Tinybird ingestion and the
vulnerabilities_listpipe to include thisreferenceLinkin the dataset/output (usinganyIfto avoid empty values).Written by Cursor Bugbot for commit 2e92aa7. This will update automatically on new commits. Configure here.