diff options
author | Otto Richter (fnetX) <git@fralix.ovh> | 2023-05-03 02:26:38 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2023-05-02 20:26:38 -0400 |
commit | 3362620ee327d48cd9ef80ec6382b84773eb1b34 (patch) | |
tree | 3b6852e2cbc789ab72d08139221c49edddd2ad02 /web_src/js/features/repo-wiki.js | |
parent | 8be6da3e2fd0b685aeb6b9e7fd9dee5a4571163a (diff) | |
download | gitea-3362620ee327d48cd9ef80ec6382b84773eb1b34.tar.gz gitea-3362620ee327d48cd9ef80ec6382b84773eb1b34.zip |
Fix api error message if fork exists (#24487)
On the @Forgejo instance of Codeberg, we discovered that forking a repo
which is already forked now returns a 500 Internal Server Error, which
is unexpected. This is an attempt at fixing this.
The error message in the log:
~~~
2023/05/02 08:36:30 .../api/v1/repo/fork.go:147:CreateFork() [E]
[6450cb8e-113] ForkRepository: repository is already forked by user
[uname: ...., repo path: .../..., fork path: .../...]
~~~
The service that is used for forking returns a custom error message
which is not checked against.
About the order of options:
The case that the fork already exists should be more common, followed by
the case that a repo with the same name already exists for other
reasons. The case that the global repo limit is hit is probably not the
likeliest.
Co-authored-by: Otto Richter <otto@codeberg.org>
Co-authored-by: Giteabot <teabot@gitea.io>
Diffstat (limited to 'web_src/js/features/repo-wiki.js')
0 files changed, 0 insertions, 0 deletions