Fix job logs endpoint: use job ID instead of job index

The GetWorkflowJobLogs endpoint was incorrectly using
DownloadActionsRunJobLogsWithIndex which expects a job index (0, 1, 2...)
but we were passing a job ID (192).

Fixed by:
- Using GetRunJobByID to get the job by its actual ID
- Using DownloadActionsRunJobLogs instead of the index-based version
- Adding validation that the job belongs to the specified run

This resolves the 404 'Job logs not found' error in CI tests.

Author: rossigee

routers/api/v1/repo/actions_run.go

@@ -682,7 +682,23 @@ func GetWorkflowJobLogs(ctx *context.APIContext) {
 
 	jobID := ctx.PathParamInt64("job_id")
 
-	if err = common.DownloadActionsRunJobLogsWithIndex(ctx.Base, ctx.Repo.Repository, runID, jobID); err != nil {
+	// Get the job by ID and verify it belongs to the run
+	job, err := actions_model.GetRunJobByID(ctx, jobID)
+	if err != nil {
+		ctx.APIErrorInternal(err)
+		return
+	}
+	if job.RunID != runID {
+		ctx.APIError(404, "Job not found in this run")
+		return
+	}
+
+	if err = job.LoadRepo(ctx); err != nil {
+		ctx.APIErrorInternal(err)
+		return
+	}
+
+	if err = common.DownloadActionsRunJobLogs(ctx.Base, ctx.Repo.Repository, job); err != nil {
 		if errors.Is(err, util.ErrNotExist) {
 			ctx.APIError(404, "Job logs not found")
 		} else {