diff options
| author | 6543 <6543@obermui.de> | 2020-04-30 19:50:47 +0200 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2020-04-30 18:50:47 +0100 |
| commit | ab69b9b1a60a83cf2b5d6c021da0cb35540df10f (patch) | |
| tree | 446be5bfe00d6e1da91f848b3ca9710fb1786645 /routers/api/v1/user | |
| parent | c25969e694245ec72c609d3be676b64361e05335 (diff) | |
| download | gitea-ab69b9b1a60a83cf2b5d6c021da0cb35540df10f.tar.gz gitea-ab69b9b1a60a83cf2b5d6c021da0cb35540df10f.zip | |
Refactor UpdateOAuth2Application (#11034)
Following on from #11008 refactor UpdateOAuth2Application
Diffstat (limited to 'routers/api/v1/user')
| -rw-r--r-- | routers/api/v1/user/app.go | 10 |
1 files changed, 2 insertions, 8 deletions
diff --git a/routers/api/v1/user/app.go b/routers/api/v1/user/app.go index f29572ef62..f426721c82 100644 --- a/routers/api/v1/user/app.go +++ b/routers/api/v1/user/app.go @@ -301,18 +301,13 @@ func UpdateOauth2Application(ctx *context.APIContext, data api.CreateOAuth2Appli // "$ref": "#/responses/OAuth2Application" appID := ctx.ParamsInt64(":id") - err := models.UpdateOAuth2Application(models.UpdateOAuth2ApplicationOptions{ + app, err := models.UpdateOAuth2Application(models.UpdateOAuth2ApplicationOptions{ Name: data.Name, UserID: ctx.User.ID, ID: appID, RedirectURIs: data.RedirectURIs, }) if err != nil { - ctx.Error(http.StatusBadRequest, "", "error updating oauth2 application") - return - } - app, err := models.GetOAuth2ApplicationByID(appID) - if err != nil { if models.IsErrOauthClientIDInvalid(err) || models.IsErrOAuthApplicationNotFound(err) { ctx.NotFound() } else { @@ -320,12 +315,11 @@ func UpdateOauth2Application(ctx *context.APIContext, data api.CreateOAuth2Appli } return } - secret, err := app.GenerateClientSecret() + app.ClientSecret, err = app.GenerateClientSecret() if err != nil { ctx.Error(http.StatusBadRequest, "", "error updating application secret") return } - app.ClientSecret = secret ctx.JSON(http.StatusOK, convert.ToOAuth2Application(app)) } |