Skip to content

Commit 84735a3

Browse files
committed
internal/scan: increase telemetry counter for show flag
Change-Id: I9bb24cb7943f8a6abd8a4c7128bfc1560b97f466 Reviewed-on: https://go-review.googlesource.com/c/vuln/+/589415 LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com> Reviewed-by: Maceo Thompson <maceothompson@google.com> Run-TryBot: Zvonimir Pavlinovic <zpavlinovic@google.com> TryBot-Result: Gopher Robot <gobot@golang.org>
1 parent fb2a687 commit 84735a3

File tree

1 file changed

+14
-3
lines changed

1 file changed

+14
-3
lines changed

internal/scan/run.go

Lines changed: 14 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -55,9 +55,7 @@ func RunGovulncheck(ctx context.Context, env []string, r io.Reader, stdout io.Wr
5555
return err
5656
}
5757

58-
counter.Inc(fmt.Sprintf("govulncheck/mode:%s", cfg.ScanMode))
59-
counter.Inc(fmt.Sprintf("govulncheck/scan:%s", cfg.ScanLevel))
60-
counter.Inc(fmt.Sprintf("govulncheck/format:%s", cfg.format))
58+
incTelemetryFlagCounters(cfg)
6159

6260
switch cfg.ScanMode {
6361
case govulncheck.ScanModeSource:
@@ -141,6 +139,19 @@ func scannerVersion(cfg *config, bi *debug.BuildInfo) {
141139
cfg.ScannerVersion = buf.String()
142140
}
143141

142+
func incTelemetryFlagCounters(cfg *config) {
143+
counter.Inc(fmt.Sprintf("govulncheck/mode:%s", cfg.ScanMode))
144+
counter.Inc(fmt.Sprintf("govulncheck/scan:%s", cfg.ScanLevel))
145+
counter.Inc(fmt.Sprintf("govulncheck/format:%s", cfg.format))
146+
147+
if len(cfg.show) == 0 {
148+
counter.Inc("govulncheck/show:none")
149+
}
150+
for _, s := range cfg.show {
151+
counter.Inc(fmt.Sprintf("govulncheck/show:%s", s))
152+
}
153+
}
154+
144155
func Flush(h govulncheck.Handler) error {
145156
if th, ok := h.(interface{ Flush() error }); ok {
146157
return th.Flush()

0 commit comments

Comments
 (0)