Skip to content

Commit 0c7e44f

Browse files
authored
Fix internal server error when updating labels without write permission (#32776) (#32785)
1 parent 3a9039b commit 0c7e44f

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

routers/api/v1/repo/issue_label.go

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -319,6 +319,11 @@ func prepareForReplaceOrAdd(ctx *context.APIContext, form api.IssueLabelsOption)
319319
return nil, nil, err
320320
}
321321

322+
if !ctx.Repo.CanWriteIssuesOrPulls(issue.IsPull) {
323+
ctx.Error(http.StatusForbidden, "CanWriteIssuesOrPulls", "write permission is required")
324+
return nil, nil, fmt.Errorf("permission denied")
325+
}
326+
322327
var (
323328
labelIDs []int64
324329
labelNames []string
@@ -350,10 +355,5 @@ func prepareForReplaceOrAdd(ctx *context.APIContext, form api.IssueLabelsOption)
350355
return nil, nil, err
351356
}
352357

353-
if !ctx.Repo.CanWriteIssuesOrPulls(issue.IsPull) {
354-
ctx.Status(http.StatusForbidden)
355-
return nil, nil, nil
356-
}
357-
358358
return issue, labels, err
359359
}

0 commit comments

Comments
 (0)