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

…ception
aquasecurity · Mar 26, 2024 · decbf40 · decbf40
2 parents 844341a + 6eab2cb
commit decbf40
Showing 1 changed file with 6 additions and 4 deletions.
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{