Minor routing and status code issues #2783

New Issue

GiteaMirror · 2025-11-02T04:48:20-06:00

GiteaMirror commented

2025-11-02 04:48:20 -06:00

Originally created by @Larivact on GitHub (Jan 20, 2019).

Gitea version (or commit ref): 1.7.0

Description

While Internal Server Errors display 500 they return a 404 status code.
Missing commits sometimes return a 500 instead of a 404 (seems to be fixed):
https://try.gitea.io/lunny/wewe/commit/000000 returns 404
https://try.gitea.io/lunny/wewe/commit/0000000 returns 500
If there are no wiki pages, any wiki page returns a 200:
https://try.gitea.io/lunny/wewe/wiki/doesnotexist
Missing wiki pages should return a 404 instead of redirecting to the index:
https://try.gitea.io/gittea/curl/wiki/doesnotexist
Invalid issue / pull requests identifiers redirect to the first issue / pull request instead of returning a 404
https://try.gitea.io/lunny/wewe/issues/doesnotexist
https://try.gitea.io/lunny/wewe/pulls/doesnotexist
/user/repo/commit/ and /user/repo/commits/ should probably redirect to the default branch.

Originally created by @Larivact on GitHub (Jan 20, 2019). - Gitea version (or commit ref): 1.7.0 ## Description * [x] While Internal Server Errors display 500 they return a 404 status code. * [x] Missing commits sometimes return a 500 instead of a 404 (seems to be fixed): https://try.gitea.io/lunny/wewe/commit/000000 returns 404 https://try.gitea.io/lunny/wewe/commit/0000000 returns 500 * [x] If there are no wiki pages, any wiki page returns a 200: https://try.gitea.io/lunny/wewe/wiki/doesnotexist * [x] Missing wiki pages should return a 404 instead of redirecting to the index: https://try.gitea.io/gittea/curl/wiki/doesnotexist * [x] Invalid issue / pull requests identifiers redirect to the first issue / pull request instead of returning a 404 https://try.gitea.io/lunny/wewe/issues/doesnotexist https://try.gitea.io/lunny/wewe/pulls/doesnotexist * [ ] `/user/repo/commit/` and `/user/repo/commits/` should probably redirect to the default branch.

GiteaMirror added the issue/confirmed type/bug labels 2025-11-02 04:48:20 -06:00

GiteaMirror closed this issue

2025-11-02 04:48:20 -06:00

GiteaMirror commented

2025-11-02 04:48:21 -06:00

@adelowo commented on GitHub (Jan 20, 2019):

💯

@adelowo commented on GitHub (Jan 20, 2019): 💯

GiteaMirror commented

2025-11-02 04:48:21 -06:00

@stale[bot] commented on GitHub (Apr 14, 2019):

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs during the next 2 weeks. Thank you for your contributions.

@stale[bot] commented on GitHub (Apr 14, 2019): This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs during the next 2 weeks. Thank you for your contributions.

GiteaMirror commented

2025-11-02 04:48:22 -06:00

@lunny commented on GitHub (Jan 28, 2021):

I think we can close this now. And I don't agree we should redirect /user/repo/commit/ to repository homepage.

@lunny commented on GitHub (Jan 28, 2021): I think we can close this now. And I don't agree we should redirect `/user/repo/commit/` to repository homepage.

GiteaMirror referenced this issue

2025-11-02 12:14:09 -06:00

[PR #2783] [MERGED] Backport to 1.2: Fix sending mail with a non-latin display name #16590

Sign in to join this conversation.

Branches Tags

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: github-starred/gitea#2783