diff options
author | John Olheiser <42128690+jolheiser@users.noreply.github.com> | 2019-03-18 21:29:43 -0500 |
---|---|---|
committer | techknowlogick <matti@mdranta.net> | 2019-03-18 22:29:43 -0400 |
commit | cac9e6e7605184f5679b1ebfbe5b5805191d9a53 (patch) | |
tree | 459a00974c8eff4f9cad8843b16d2a1f476c5912 /routers/api/v1/org | |
parent | d10a668ffc4ed2a81a8a62ee78f0885ede713ddd (diff) | |
download | gitea-cac9e6e7605184f5679b1ebfbe5b5805191d9a53.tar.gz gitea-cac9e6e7605184f5679b1ebfbe5b5805191d9a53.zip |
Updates to API 404 responses (#6077)
Diffstat (limited to 'routers/api/v1/org')
-rw-r--r-- | routers/api/v1/org/hook.go | 2 | ||||
-rw-r--r-- | routers/api/v1/org/member.go | 6 | ||||
-rw-r--r-- | routers/api/v1/org/team.go | 4 |
3 files changed, 6 insertions, 6 deletions
diff --git a/routers/api/v1/org/hook.go b/routers/api/v1/org/hook.go index 84559a958e..5825de2c1c 100644 --- a/routers/api/v1/org/hook.go +++ b/routers/api/v1/org/hook.go @@ -164,7 +164,7 @@ func DeleteHook(ctx *context.APIContext) { hookID := ctx.ParamsInt64(":id") if err := models.DeleteWebhookByOrgID(org.ID, hookID); err != nil { if models.IsErrWebhookNotExist(err) { - ctx.Status(404) + ctx.NotFound() } else { ctx.Error(500, "DeleteWebhookByOrgID", err) } diff --git a/routers/api/v1/org/member.go b/routers/api/v1/org/member.go index b66d86c660..33c906ec6a 100644 --- a/routers/api/v1/org/member.go +++ b/routers/api/v1/org/member.go @@ -135,11 +135,11 @@ func IsMember(ctx *context.APIContext) { } else if userToCheckIsMember { ctx.Status(204) } else { - ctx.Status(404) + ctx.NotFound() } return } else if ctx.User.ID == userToCheck.ID { - ctx.Status(404) + ctx.NotFound() return } } @@ -177,7 +177,7 @@ func IsPublicMember(ctx *context.APIContext) { if userToCheck.IsPublicMember(ctx.Org.Organization.ID) { ctx.Status(204) } else { - ctx.Status(404) + ctx.NotFound() } } diff --git a/routers/api/v1/org/team.go b/routers/api/v1/org/team.go index a1916db00b..ca3de9f04e 100644 --- a/routers/api/v1/org/team.go +++ b/routers/api/v1/org/team.go @@ -242,7 +242,7 @@ func GetTeamMembers(ctx *context.APIContext) { ctx.Error(500, "IsOrganizationMember", err) return } else if !isMember { - ctx.Status(404) + ctx.NotFound() return } team := ctx.Org.Team @@ -391,7 +391,7 @@ func getRepositoryByParams(ctx *context.APIContext) *models.Repository { repo, err := models.GetRepositoryByName(ctx.Org.Team.OrgID, ctx.Params(":reponame")) if err != nil { if models.IsErrRepoNotExist(err) { - ctx.Status(404) + ctx.NotFound() } else { ctx.Error(500, "GetRepositoryByName", err) } |