internal/govulncheck: fix error value for compare finding

Otherwise, unmarshaling fails. The marshaling and unmarshaling are
needed to communicate results from the govulncheck compare sandbox. We
can marshal the error type, but unmarshaling to an interface type will
fail if the underlying type is not one of the supported ones:
https://pkg.go.dev/encoding/json#Unmarshal.

This CL also propagates some errors to the top for easier monitoring.

Change-Id: I72d1123d75cda40959e323cf5d241375fc8321c6
Reviewed-on: https://go-review.googlesource.com/c/pkgsite-metrics/+/523815
Reviewed-by: Jonathan Amsterdam <jba@google.com>
TryBot-Result: Gopher Robot <gobot@golang.org>
Run-TryBot: Zvonimir Pavlinovic <zpavlinovic@google.com>
3 files changed
tree: c1a3cac93e158f424fcb9aca0d8fcee14e82702c
  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.