Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

util/errors: Use "403 Forbidden" for GitHub permission errors #8028

Merged
merged 1 commit into from
Jan 30, 2024

Conversation

Turbo87
Copy link
Member

@Turbo87 Turbo87 commented Jan 30, 2024

I guess it's debatable which status code exactly we should use here, but this should definitely not use "200 OK" 😅

Related:

@Turbo87 Turbo87 added C-enhancement ✨ Category: Adding new behavior or a change to the way an existing feature works A-backend ⚙️ labels Jan 30, 2024
@Turbo87 Turbo87 enabled auto-merge (squash) January 30, 2024 14:26
@Turbo87 Turbo87 merged commit c59eaed into rust-lang:main Jan 30, 2024
6 checks passed
@Turbo87 Turbo87 deleted the github-errors branch January 30, 2024 14:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-backend ⚙️ C-enhancement ✨ Category: Adding new behavior or a change to the way an existing feature works
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant