fix: use tailLines if bigger but only if <= 5000

mattdholloway · mattdholloway · commit 516c0f731cc3 · 2025-08-18T13:41:24.000+01:00
diff --git a/pkg/github/actions.go b/pkg/github/actions.go
@@ -761,9 +761,9 @@ func downloadLogContent(ctx context.Context, logURL string, tailLines int, maxLi
 		tailLines = 1000
 	}
 
-	bufferSize := tailLines
-	if bufferSize > maxLines {
-		bufferSize = maxLines
+	bufferSize := maxLines
+	if tailLines > maxLines && tailLines <= 5000 {
+		bufferSize = tailLines
 	}
 
 	processedInput, totalLines, httpResp, err := buffer.ProcessResponseAsRingBufferToEnd(httpResp, bufferSize)

Original file line number	Diff line number	Diff line change
`@@ -761,9 +761,9 @@ func downloadLogContent(ctx context.Context, logURL string, tailLines int, maxLi`
`761`	`761`	`tailLines = 1000`
`762`	`762`	`}`
`763`	`763`
`764`		`- bufferSize := tailLines`
`765`		`- if bufferSize > maxLines {`
`766`		`- bufferSize = maxLines`
	`764`	`+ bufferSize := maxLines`
	`765`	`+ if tailLines > maxLines && tailLines <= 5000 {`
	`766`	`+ bufferSize = tailLines`
`767`	`767`	`}`
`768`	`768`
`769`	`769`	`processedInput, totalLines, httpResp, err := buffer.ProcessResponseAsRingBufferToEnd(httpResp, bufferSize)`