ossf/scorecard
1
1
Fork 0
mirror of https://github.com/ossf/scorecard.git synced 2024-11-13 01:25:07 +03:00
Code Issues Actions 6 Packages Projects Releases Wiki Activity

changes (#1062)

Browse Source
This commit is contained in:
laurentsimon 2021-09-27 08:50:58 -07:00 committed by GitHub
parent 2c16597b51
commit 6b9010e575
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 41 additions and 17 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

58
cmd/root.go
View File

@ -72,11 +72,13 @@ func readPolicy() (*spol.ScorecardPolicy, error) {
if policyFile != "" {
data, err := os.ReadFile(policyFile)
if err != nil {
return nil, fmt.Errorf("os.ReadFile: %w", err)
return nil, sce.WithMessage(sce.ErrScorecardInternal,
fmt.Sprintf("os.ReadFile: %v", err))
}
sp, err := spol.ParseFromYAML(data)
if err != nil {
return nil, fmt.Errorf("spol.ParseFromYAML: %w", err)
return nil,
sce.WithMessage(sce.ErrScorecardInternal, fmt.Sprintf("spol.ParseFromYAML: %v", err))
}
return sp, nil
}
@ -94,6 +96,39 @@ func checksHavePolicies(sp *spol.ScorecardPolicy, enabledChecks checker.CheckNam
return true
}
func getEnabledChecks(sp *spol.ScorecardPolicy, argsChecks []string) (checker.CheckNameToFnMap, error) {
enabledChecks := checker.CheckNameToFnMap{}
switch {
case len(argsChecks) != 0:
// Populate checks to run with the CLI arguments.
for _, checkName := range argsChecks {
if !enableCheck(checkName, &enabledChecks) {
return enabledChecks,
sce.WithMessage(sce.ErrScorecardInternal, fmt.Sprintf("Invalid check: %s", checkName))
}
}
case sp != nil:
// Populate checks to run with policy file.
for checkName := range sp.GetPolicies() {
if !enableCheck(checkName, &enabledChecks) {
return enabledChecks,
sce.WithMessage(sce.ErrScorecardInternal, fmt.Sprintf("Invalid check: %s", checkName))
}
}
default:
enabledChecks = checks.AllChecks
}
// If a policy was passed as argument, ensure all checks
// to run have a corresponding policy.
if sp != nil && !checksHavePolicies(sp, enabledChecks) {
return enabledChecks, sce.WithMessage(sce.ErrScorecardInternal, "checks don't have policies")
}
return enabledChecks, nil
}
var rootCmd = &cobra.Command{
Use: scorecardUse,
Short: scorecardShort,
@ -151,28 +186,17 @@ var rootCmd = &cobra.Command{
log.Fatal(err)
}
enabledChecks := checker.CheckNameToFnMap{}
if len(checksToRun) != 0 {
for _, checkToRun := range checksToRun {
if !enableCheck(checkToRun, &enabledChecks) {
log.Fatalf("Invalid check: %s", checkToRun)
}
}
} else {
enabledChecks = checks.AllChecks
enabledChecks, err := getEnabledChecks(policy, checksToRun)
if err != nil {
log.Fatal(err)
}
if format == formatDefault {
for checkName := range enabledChecks {
fmt.Fprintf(os.Stderr, "Starting [%s]\n", checkName)
}
}
// If a policy was passed as argument, ensure all checks
// to run have a corresponding policy.
if policy != nil && !checksHavePolicies(policy, enabledChecks) {
log.Fatal("checks don't have policies")
}
ctx := context.Background()
logger, err := githubrepo.NewLogger(*logLevel)