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

x/vulndb: improve internal/proxy package #60275

Open
tatianab opened this issue May 17, 2023 · 2 comments
Open

x/vulndb: improve internal/proxy package #60275

tatianab opened this issue May 17, 2023 · 2 comments
Assignees
Labels
vulncheck or vulndb Issues for the x/vuln or x/vulndb repo

Comments

@tatianab
Copy link

  • Refactor to make it easier to test
  • Add unit tests
  • Cache proxy lookups
@tatianab tatianab added the vulncheck or vulndb Issues for the x/vuln or x/vulndb repo label May 17, 2023
@tatianab tatianab self-assigned this May 17, 2023
@gopherbot gopherbot modified the milestones: Unreleased, vuln/unplanned May 17, 2023
@gopherbot
Copy link

Change https://go.dev/cl/497036 mentions this issue: internal/proxy: add unit tests for proxy package

gopherbot pushed a commit to golang/vulndb that referenced this issue May 23, 2023
Adds additional unit tests for the proxy package.

For golang/go#60275

Change-Id: I4706cb2c0e3b54db27241eb19177bd48d2683fba
Reviewed-on: https://go-review.googlesource.com/c/vulndb/+/497036
Reviewed-by: Damien Neil <dneil@google.com>
Run-TryBot: Tatiana Bradley <tatianabradley@google.com>
TryBot-Result: Gopher Robot <gobot@golang.org>
@gopherbot
Copy link

Change https://go.dev/cl/506316 mentions this issue: internal/proxy: cache proxy lookups

gopherbot pushed a commit to golang/vulndb that referenced this issue Jun 26, 2023
Cache proxy lookups with a simple in-memory cache.

For golang/go#60275

Change-Id: I24016f3d78b7fb70d452db5286abae0e4e71ba5a
Reviewed-on: https://go-review.googlesource.com/c/vulndb/+/506316
Reviewed-by: Damien Neil <dneil@google.com>
TryBot-Result: Gopher Robot <gobot@golang.org>
Run-TryBot: Tatiana Bradley <tatianabradley@google.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
vulncheck or vulndb Issues for the x/vuln or x/vulndb repo
Projects
None yet
Development

No branches or pull requests

2 participants