Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

dependencies: enhance CVE and version links. #13434

Merged
merged 1 commit into from
Oct 8, 2020

Conversation

htuch
Copy link
Member

@htuch htuch commented Oct 8, 2020

  • CPE link now performs NIST search on all CVEs, rather than linking to
    a collection of version CPEs and requiring the user to click on each
    for CVEs.

  • Version links now point to GitHub release tag pages or the GitHub
    tree at a particular hash for SHA versions. Previously this was just
    the tarball download. For non-GitHub blobs, e.g. on GCS, we still just
    provide the tarball download link.

Risk level: Low
Testing: Manual clicking on links, seems to work for NIST CVEs and a
wide number of sampled tagged release versions.

Part of #12673

Signed-off-by: Harvey Tuch htuch@google.com

* CPE link now performs NIST search on all CVEs, rather than linking to
  a collection of version CPEs and requiring the user to click on each
  for CVEs.

* Version links now point to GitHub release tag pages or the GitHub
  tree at a particular hash for SHA versions. Previously this was just
  the tarball download. For non-GitHub blobs, e.g. on GCS, we still just
  provide the tarball download link.

Risk level: Low
Testing: Manual clicking on links, seems to work for NIST CVEs and a
  wide number of sampled tagged release versions.

Part of envoyproxy#12673

Signed-off-by: Harvey Tuch <htuch@google.com>
@moderation
Copy link
Contributor

Ran a test and LGTM. Good to not link directly to the tarball (where possible)

Copy link
Member

@mattklein123 mattklein123 left a comment

Choose a reason for hiding this comment

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

Neat!

@htuch htuch merged commit c9aca3c into envoyproxy:master Oct 8, 2020
@htuch htuch deleted the dashboard-links branch October 8, 2020 17:32
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