Skip to content

Commit

Permalink
fix: remove security tracker URL validation to reduce job runtime (#2481
Browse files Browse the repository at this point in the history
)

#2388 significantly increased
`combine-to-osv` runtime and caused job failures. Remove the URL
validation to reduce time.

Follow up PR: run all HTTP request parallelly.
  • Loading branch information
hogo6002 authored Aug 16, 2024
1 parent 02575ef commit ccce244
Showing 1 changed file with 0 additions and 5 deletions.
5 changes: 0 additions & 5 deletions vulnfeeds/cmd/combine-to-osv/main.go
Original file line number Diff line number Diff line change
Expand Up @@ -253,10 +253,5 @@ func addReference(cveId string, ecosystem string, convertedCve *vulns.Vulnerabil
return
}

_, err := cves.ValidateAndCanonicalizeLink(securityReference.URL)
if err != nil {
Logger.Warnf("Failed to add reference for %s in %s: %v", cveId, ecosystem, err)
return
}
convertedCve.References = append(convertedCve.References, securityReference)
}

0 comments on commit ccce244

Please sign in to comment.