Merge pull request #77 from aquasecurity/bugfix/azure-out-of-range-exception

davidsalame1 · web-flow · commit decbf405ce8d · 2024-03-26T13:21:39.000+02:00
diff --git a/pkg/commenter/azure/azure.go b/pkg/commenter/azure/azure.go
@@ -73,12 +73,14 @@ func (c *Azure) WriteMultiLineComment(file, comment string, startLine, endLine i
 		file = fmt.Sprintf("/%s", file)
 	}
 
-	if startLine == commenter.FIRST_AVAILABLE_LINE {
-		startLine = 0
+	if startLine == commenter.FIRST_AVAILABLE_LINE || startLine == 0 {
+		// Reference: https://developercommunity.visualstudio.com/t/Adding-thread-to-PR-using-REST-API-cause/10598424
+		startLine = 1
 	}
 
-	if endLine == commenter.FIRST_AVAILABLE_LINE {
-		endLine = 0
+	if endLine == commenter.FIRST_AVAILABLE_LINE || endLine == 0 {
+		// Reference: https://developercommunity.visualstudio.com/t/Adding-thread-to-PR-using-REST-API-cause/10598424
+		endLine = 1
 	}
 
 	b := Body{

Original file line number	Diff line number	Diff line change
`@@ -73,12 +73,14 @@ func (c *Azure) WriteMultiLineComment(file, comment string, startLine, endLine i`
`73`	`73`	`file = fmt.Sprintf("/%s", file)`
`74`	`74`	`}`
`75`	`75`
`76`		`- if startLine == commenter.FIRST_AVAILABLE_LINE {`
`77`		`- startLine = 0`
	`76`	`+ if startLine == commenter.FIRST_AVAILABLE_LINE \|\| startLine == 0 {`
	`77`	`+ // Reference: https://developercommunity.visualstudio.com/t/Adding-thread-to-PR-using-REST-API-cause/10598424`
	`78`	`+ startLine = 1`
`78`	`79`	`}`
`79`	`80`
`80`		`- if endLine == commenter.FIRST_AVAILABLE_LINE {`
`81`		`- endLine = 0`
	`81`	`+ if endLine == commenter.FIRST_AVAILABLE_LINE \|\| endLine == 0 {`
	`82`	`+ // Reference: https://developercommunity.visualstudio.com/t/Adding-thread-to-PR-using-REST-API-cause/10598424`
	`83`	`+ endLine = 1`
`82`	`84`	`}`
`83`	`85`
`84`	`86`	`b := Body{`