diff --git a/internal/check/valid_owner.go b/internal/check/valid_owner.go index b5b7d8a..cbe06f5 100644 --- a/internal/check/valid_owner.go +++ b/internal/check/valid_owner.go @@ -375,6 +375,7 @@ func (*ValidOwner) checkRequiredScopes(header http.Header) error { gotScopes := strings.Split(header.Get(scopeHeader), ",") presentScope := map[github.Scope]struct{}{} for _, scope := range gotScopes { + scope = strings.TrimSpace(scope) presentScope[github.Scope(scope)] = struct{}{} } diff --git a/internal/runner/runner_worker.go b/internal/runner/runner_worker.go index 39074d1..49dce3c 100644 --- a/internal/runner/runner_worker.go +++ b/internal/runner/runner_worker.go @@ -95,6 +95,10 @@ func (r *CheckRunner) collectMetrics(checkOut check.Output, err error) { r.allFoundIssues[i.Severity]++ } + if err != nil { + r.allFoundIssues[check.Error]++ + } + if len(checkOut.Issues) > 0 || err != nil { r.notPassedChecksCnt++ }