internal/worker: remove binary mode from govulncheck

Binary analysis is done as part of compare mode, so we don't have to
support binary mode independently (which is anyhow analyzing only one
binary).

This CL also makes few simple related refactorings:
 - make explicit that govulncheck_sandbox will not run binary analysis
 - remove dependency of govulncheck_sandbox on internal/worker. The
   latter actually depends on the former, so the only reason why this
   compiled in the past is because worker called sandbox via command
   line

This should result in code simplification.

Change-Id: Ia6374904f78c21f240fe5a769a21e349f68576cc
Reviewed-on: https://go-review.googlesource.com/c/pkgsite-metrics/+/527978
Reviewed-by: Maceo Thompson <maceothompson@google.com>
Run-TryBot: Zvonimir Pavlinovic <zpavlinovic@google.com>
TryBot-Result: Gopher Robot <gobot@golang.org>
Reviewed-by: Jonathan Amsterdam <jba@google.com>
6 files changed
tree: 246ddae91e3b72bab5b959452994a1656a7f0a72
  1. cmd/
  2. deploy/
  3. devtools/
  4. internal/
  5. terraform/
  6. .dockerignore
  7. .gitignore
  8. all_test.go
  9. checks.bash
  10. config.json.commented
  11. CONTRIBUTING.md
  12. go.mod
  13. go.sum
  14. LICENSE
  15. Makefile
  16. PATENTS
  17. README.md
  18. tools.go
README.md

pkgsite-metrics

This repository contains code that serves pkg.go.dev/metrics.

Report Issues / Send Patches

This repository uses Gerrit for code changes. To learn how to submit changes to this repository, see https://golang.org/doc/contribute.html.

The main issue tracker for the time repository is located at https://github.com/golang/go/issues. Prefix your issue with “x/pkgsite-metrics:” in the subject line, so it is easy to find.