Fix bug on downloading job logs

lunny · lunny · commit acbb69cfadb3 · 2025-03-27T12:31:56.000-07:00
diff --git a/routers/common/actions.go b/routers/common/actions.go
@@ -22,7 +22,7 @@ func DownloadActionsRunJobLogsWithIndex(ctx *context.Base, ctxRepo *repo_model.R
 	if err = runJobs.LoadRepos(ctx); err != nil {
 		return fmt.Errorf("LoadRepos: %w", err)
 	}
-	if 0 < jobIndex || jobIndex >= int64(len(runJobs)) {
+	if jobIndex < 0 || jobIndex >= int64(len(runJobs)) {
 		return util.NewNotExistErrorf("job index is out of range: %d", jobIndex)
 	}
 	return DownloadActionsRunJobLogs(ctx, ctxRepo, runJobs[jobIndex])

Original file line number	Diff line number	Diff line change
`@@ -22,7 +22,7 @@ func DownloadActionsRunJobLogsWithIndex(ctx context.Base, ctxRepo repo_model.R`
`22`	`22`	`if err = runJobs.LoadRepos(ctx); err != nil {`
`23`	`23`	`return fmt.Errorf("LoadRepos: %w", err)`
`24`	`24`	`}`
`25`		`- if 0 < jobIndex \|\| jobIndex >= int64(len(runJobs)) {`
	`25`	`+ if jobIndex < 0 \|\| jobIndex >= int64(len(runJobs)) {`
`26`	`26`	`return util.NewNotExistErrorf("job index is out of range: %d", jobIndex)`
`27`	`27`	`}`
`28`	`28`	`return DownloadActionsRunJobLogs(ctx, ctxRepo, runJobs[jobIndex])`