mirror of https://github.com/go-gitea/gitea.git
Check user instead of organization when creating a repo from a template via API (#16346)
* Check user instead of organization * Enforce that only admins can copy a repo to another user
This commit is contained in:
parent
ff69dfff7a
commit
251d7f524a
|
@ -374,16 +374,21 @@ func Generate(ctx *context.APIContext) {
|
||||||
ctxUser := ctx.User
|
ctxUser := ctx.User
|
||||||
var err error
|
var err error
|
||||||
if form.Owner != ctxUser.Name {
|
if form.Owner != ctxUser.Name {
|
||||||
ctxUser, err = models.GetOrgByName(form.Owner)
|
ctxUser, err = models.GetUserByName(form.Owner)
|
||||||
if err != nil {
|
if err != nil {
|
||||||
if models.IsErrOrgNotExist(err) {
|
if models.IsErrUserNotExist(err) {
|
||||||
ctx.JSON(http.StatusNotFound, map[string]interface{}{
|
ctx.JSON(http.StatusNotFound, map[string]interface{}{
|
||||||
"error": "request owner `" + form.Name + "` is not exist",
|
"error": "request owner `" + form.Owner + "` does not exist",
|
||||||
})
|
})
|
||||||
return
|
return
|
||||||
}
|
}
|
||||||
|
|
||||||
ctx.Error(http.StatusInternalServerError, "GetOrgByName", err)
|
ctx.Error(http.StatusInternalServerError, "GetUserByName", err)
|
||||||
|
return
|
||||||
|
}
|
||||||
|
|
||||||
|
if !ctx.User.IsAdmin && !ctxUser.IsOrganization() {
|
||||||
|
ctx.Error(http.StatusForbidden, "", "Only admin can generate repository for other user.")
|
||||||
return
|
return
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue