Merge pull request #720 from opendevstack/fix/report-sonar-scanner-stderr

Michael Sauter · web-flow · commit 35dce3b5b32c · 2023-07-21T13:37:56.000+02:00
report sonar-scanner stderr
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -14,6 +14,8 @@ listed in the changelog.
 
 - ods.yaml branch trigger patterns must be lowercase ([#713](https://github.com/opendevstack/ods-pipeline/issues/713))
 
+- sonar-scanner invocations stderr not captured ([#719](https://github.com/opendevstack/ods-pipeline/issues/719))
+
 ## [0.13.2] - 2023-07-18
 
 ### Fixed
diff --git a/cmd/sonar/main.go b/cmd/sonar/main.go
@@ -119,7 +119,7 @@ func sonarScan(
 		ctxt.GitCommitSHA,
 		prInfo,
 		os.Stdout,
-		os.Stdin,
+		os.Stderr,
 	)
 	if err != nil {
 		return fmt.Errorf("scan failed: %w", err)

Original file line number	Diff line number	Diff line change
`@@ -119,7 +119,7 @@ func sonarScan(`
`119`	`119`	`ctxt.GitCommitSHA,`
`120`	`120`	`prInfo,`
`121`	`121`	`os.Stdout,`
`122`		`- os.Stdin,`
	`122`	`+ os.Stderr,`
`123`	`123`	`)`
`124`	`124`	`if err != nil {`
`125`	`125`	`return fmt.Errorf("scan failed: %w", err)`