a
/
agola
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

api: use httpError helper everywhere

This commit is contained in:
Simone Gotti 2019-04-05 16:33:00 +02:00
parent fe5bc2fa31
commit 595590e611
17 changed files with 190 additions and 112 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
internal/services/configstore/api/org.go
View File

@ -49,7 +49,7 @@ func (h *OrgHandler) ServeHTTP(w http.ResponseWriter, r *http.Request) {
}) })
if err != nil { if err != nil {
h.log.Errorf("err: %+v", err) h.log.Errorf("err: %+v", err)
http.Error(w, err.Error(), http.StatusInternalServerError) httpError(w, err)
return return
} }
@ -60,7 +60,7 @@ func (h *OrgHandler) ServeHTTP(w http.ResponseWriter, r *http.Request) {
if err := json.NewEncoder(w).Encode(org); err != nil { if err := json.NewEncoder(w).Encode(org); err != nil {
h.log.Errorf("err: %+v", err) h.log.Errorf("err: %+v", err)
http.Error(w, err.Error(), http.StatusInternalServerError) httpError(w, err)
return return
} }
} }
@ -86,7 +86,7 @@ func (h *OrgByNameHandler) ServeHTTP(w http.ResponseWriter, r *http.Request) {
}) })
if err != nil { if err != nil {
h.log.Errorf("err: %+v", err) h.log.Errorf("err: %+v", err)
http.Error(w, err.Error(), http.StatusInternalServerError) httpError(w, err)
return return
} }
@ -97,7 +97,7 @@ func (h *OrgByNameHandler) ServeHTTP(w http.ResponseWriter, r *http.Request) {
if err := json.NewEncoder(w).Encode(org); err != nil { if err := json.NewEncoder(w).Encode(org); err != nil {
h.log.Errorf("err: %+v", err) h.log.Errorf("err: %+v", err)
http.Error(w, err.Error(), http.StatusInternalServerError) httpError(w, err)
return return
} }
} }
@ -129,7 +129,7 @@ func (h *CreateOrgHandler) ServeHTTP(w http.ResponseWriter, r *http.Request) {
if err := json.NewEncoder(w).Encode(org); err != nil { if err := json.NewEncoder(w).Encode(org); err != nil {
h.log.Errorf("err: %+v", err) h.log.Errorf("err: %+v", err)
http.Error(w, err.Error(), http.StatusInternalServerError) httpError(w, err)
return return
} }
} }
@ -205,13 +205,13 @@ func (h *OrgsHandler) ServeHTTP(w http.ResponseWriter, r *http.Request) {
}) })
if err != nil { if err != nil {
h.log.Errorf("err: %+v", err) h.log.Errorf("err: %+v", err)
http.Error(w, err.Error(), http.StatusInternalServerError) httpError(w, err)
return return
} }
if err := json.NewEncoder(w).Encode(orgs); err != nil { if err := json.NewEncoder(w).Encode(orgs); err != nil {
h.log.Errorf("err: %+v", err) h.log.Errorf("err: %+v", err)
http.Error(w, err.Error(), http.StatusInternalServerError) httpError(w, err)
return return
} }
} }

9
internal/services/configstore/api/project.go
View File

@ -64,7 +64,8 @@ func (h *ProjectHandler) ServeHTTP(w http.ResponseWriter, r *http.Request) {
return err return err
}) })
if err != nil { if err != nil {
http.Error(w, err.Error(), http.StatusInternalServerError) h.log.Errorf("err: %+v", err)
httpError(w, err)
return return
} }
@ -74,7 +75,8 @@ func (h *ProjectHandler) ServeHTTP(w http.ResponseWriter, r *http.Request) {
} }
if err := json.NewEncoder(w).Encode(project); err != nil { if err := json.NewEncoder(w).Encode(project); err != nil {
http.Error(w, err.Error(), http.StatusInternalServerError) h.log.Errorf("err: %+v", err)
httpError(w, err)
return return
} }
} }
@ -106,7 +108,8 @@ func (h *CreateProjectHandler) ServeHTTP(w http.ResponseWriter, r *http.Request)
} }
if err := json.NewEncoder(w).Encode(project); err != nil { if err := json.NewEncoder(w).Encode(project); err != nil {
http.Error(w, err.Error(), http.StatusInternalServerError) h.log.Errorf("err: %+v", err)
httpError(w, err)
return return
} }
} }

27
internal/services/configstore/api/projectgroup.go
View File

@ -52,7 +52,8 @@ func (h *ProjectGroupHandler) ServeHTTP(w http.ResponseWriter, r *http.Request)
return err return err
}) })
if err != nil { if err != nil {
http.Error(w, err.Error(), http.StatusInternalServerError) h.log.Errorf("err: %+v", err)
httpError(w, err)
return return
} }
@ -62,7 +63,8 @@ func (h *ProjectGroupHandler) ServeHTTP(w http.ResponseWriter, r *http.Request)
} }
if err := json.NewEncoder(w).Encode(projectGroup); err != nil { if err := json.NewEncoder(w).Encode(projectGroup); err != nil {
http.Error(w, err.Error(), http.StatusInternalServerError) h.log.Errorf("err: %+v", err)
httpError(w, err)
return return
} }
} }
@ -90,7 +92,8 @@ func (h *ProjectGroupProjectsHandler) ServeHTTP(w http.ResponseWriter, r *http.R
return err return err
}) })
if err != nil { if err != nil {
http.Error(w, err.Error(), http.StatusInternalServerError) h.log.Errorf("err: %+v", err)
httpError(w, err)
return return
} }
@ -106,12 +109,14 @@ func (h *ProjectGroupProjectsHandler) ServeHTTP(w http.ResponseWriter, r *http.R
return err return err
}) })
if err != nil { if err != nil {
http.Error(w, err.Error(), http.StatusInternalServerError) h.log.Errorf("err: %+v", err)
httpError(w, err)
return return
} }
if err := json.NewEncoder(w).Encode(projects); err != nil { if err := json.NewEncoder(w).Encode(projects); err != nil {
http.Error(w, err.Error(), http.StatusInternalServerError) h.log.Errorf("err: %+v", err)
httpError(w, err)
return return
} }
} }
@ -139,7 +144,8 @@ func (h *ProjectGroupSubgroupsHandler) ServeHTTP(w http.ResponseWriter, r *http.
return err return err
}) })
if err != nil { if err != nil {
http.Error(w, err.Error(), http.StatusInternalServerError) h.log.Errorf("err: %+v", err)
httpError(w, err)
return return
} }
@ -155,12 +161,14 @@ func (h *ProjectGroupSubgroupsHandler) ServeHTTP(w http.ResponseWriter, r *http.
return err return err
}) })
if err != nil { if err != nil {
http.Error(w, err.Error(), http.StatusInternalServerError) h.log.Errorf("err: %+v", err)
httpError(w, err)
return return
} }
if err := json.NewEncoder(w).Encode(projectGroups); err != nil { if err := json.NewEncoder(w).Encode(projectGroups); err != nil {
http.Error(w, err.Error(), http.StatusInternalServerError) h.log.Errorf("err: %+v", err)
httpError(w, err)
return return
} }
} }
@ -192,7 +200,8 @@ func (h *CreateProjectGroupHandler) ServeHTTP(w http.ResponseWriter, r *http.Req
} }
if err := json.NewEncoder(w).Encode(projectGroup); err != nil { if err := json.NewEncoder(w).Encode(projectGroup); err != nil {
http.Error(w, err.Error(), http.StatusInternalServerError) h.log.Errorf("err: %+v", err)
httpError(w, err)
return return
} }
} }

21
internal/services/configstore/api/remotesource.go
View File

@ -48,7 +48,8 @@ func (h *RemoteSourceHandler) ServeHTTP(w http.ResponseWriter, r *http.Request)
return err return err
}) })
if err != nil { if err != nil {
http.Error(w, err.Error(), http.StatusInternalServerError) h.log.Errorf("err: %+v", err)
httpError(w, err)
return return
} }
@ -58,7 +59,8 @@ func (h *RemoteSourceHandler) ServeHTTP(w http.ResponseWriter, r *http.Request)
} }
if err := json.NewEncoder(w).Encode(remoteSource); err != nil { if err := json.NewEncoder(w).Encode(remoteSource); err != nil {
http.Error(w, err.Error(), http.StatusInternalServerError) h.log.Errorf("err: %+v", err)
httpError(w, err)
return return
} }
} }
@ -83,7 +85,8 @@ func (h *RemoteSourceByNameHandler) ServeHTTP(w http.ResponseWriter, r *http.Req
return err return err
}) })
if err != nil { if err != nil {
http.Error(w, err.Error(), http.StatusInternalServerError) h.log.Errorf("err: %+v", err)
httpError(w, err)
return return
} }
@ -93,7 +96,8 @@ func (h *RemoteSourceByNameHandler) ServeHTTP(w http.ResponseWriter, r *http.Req
} }
if err := json.NewEncoder(w).Encode(remoteSource); err != nil { if err := json.NewEncoder(w).Encode(remoteSource); err != nil {
http.Error(w, err.Error(), http.StatusInternalServerError) h.log.Errorf("err: %+v", err)
httpError(w, err)
return return
} }
} }
@ -125,7 +129,8 @@ func (h *CreateRemoteSourceHandler) ServeHTTP(w http.ResponseWriter, r *http.Req
} }
if err := json.NewEncoder(w).Encode(remoteSource); err != nil { if err := json.NewEncoder(w).Encode(remoteSource); err != nil {
http.Error(w, err.Error(), http.StatusInternalServerError) h.log.Errorf("err: %+v", err)
httpError(w, err)
return return
} }
} }
@ -194,12 +199,14 @@ func (h *RemoteSourcesHandler) ServeHTTP(w http.ResponseWriter, r *http.Request)
remoteSources, err := h.readDB.GetRemoteSources(start, limit, asc) remoteSources, err := h.readDB.GetRemoteSources(start, limit, asc)
if err != nil { if err != nil {
http.Error(w, err.Error(), http.StatusInternalServerError) h.log.Errorf("err: %+v", err)
httpError(w, err)
return return
} }
if err := json.NewEncoder(w).Encode(remoteSources); err != nil { if err := json.NewEncoder(w).Encode(remoteSources); err != nil {
http.Error(w, err.Error(), http.StatusInternalServerError) h.log.Errorf("err: %+v", err)
httpError(w, err)
return return
} }
} }

15
internal/services/configstore/api/secret.go
View File

@ -47,7 +47,8 @@ func (h *SecretHandler) ServeHTTP(w http.ResponseWriter, r *http.Request) {
return err return err
}) })
if err != nil { if err != nil {
http.Error(w, err.Error(), http.StatusInternalServerError) h.log.Errorf("err: %+v", err)
httpError(w, err)
return return
} }
@ -57,7 +58,8 @@ func (h *SecretHandler) ServeHTTP(w http.ResponseWriter, r *http.Request) {
} }
if err := json.NewEncoder(w).Encode(secret); err != nil { if err := json.NewEncoder(w).Encode(secret); err != nil {
http.Error(w, err.Error(), http.StatusInternalServerError) h.log.Errorf("err: %+v", err)
httpError(w, err)
return return
} }
} }
@ -104,12 +106,14 @@ func (h *SecretsHandler) ServeHTTP(w http.ResponseWriter, r *http.Request) {
return err return err
}) })
if err != nil { if err != nil {
http.Error(w, err.Error(), http.StatusInternalServerError) h.log.Errorf("err: %+v", err)
httpError(w, err)
return return
} }
if err := json.NewEncoder(w).Encode(secrets); err != nil { if err := json.NewEncoder(w).Encode(secrets); err != nil {
http.Error(w, err.Error(), http.StatusInternalServerError) h.log.Errorf("err: %+v", err)
httpError(w, err)
return return
} }
} }
@ -149,7 +153,8 @@ func (h *CreateSecretHandler) ServeHTTP(w http.ResponseWriter, r *http.Request)
} }
if err := json.NewEncoder(w).Encode(secret); err != nil { if err := json.NewEncoder(w).Encode(secret); err != nil {
http.Error(w, err.Error(), http.StatusInternalServerError) h.log.Errorf("err: %+v", err)
httpError(w, err)
return return
} }
} }

26
internal/services/configstore/api/user.go
View File

@ -50,7 +50,7 @@ func (h *UserHandler) ServeHTTP(w http.ResponseWriter, r *http.Request) {
}) })
if err != nil { if err != nil {
h.log.Errorf("err: %+v", err) h.log.Errorf("err: %+v", err)
http.Error(w, err.Error(), http.StatusInternalServerError) httpError(w, err)
return return
} }
@ -61,7 +61,7 @@ func (h *UserHandler) ServeHTTP(w http.ResponseWriter, r *http.Request) {
if err := json.NewEncoder(w).Encode(user); err != nil { if err := json.NewEncoder(w).Encode(user); err != nil {
h.log.Errorf("err: %+v", err) h.log.Errorf("err: %+v", err)
http.Error(w, err.Error(), http.StatusInternalServerError) httpError(w, err)
return return
} }
} }
@ -87,7 +87,7 @@ func (h *UserByNameHandler) ServeHTTP(w http.ResponseWriter, r *http.Request) {
}) })
if err != nil { if err != nil {
h.log.Errorf("err: %+v", err) h.log.Errorf("err: %+v", err)
http.Error(w, err.Error(), http.StatusInternalServerError) httpError(w, err)
return return
} }
@ -98,7 +98,7 @@ func (h *UserByNameHandler) ServeHTTP(w http.ResponseWriter, r *http.Request) {
if err := json.NewEncoder(w).Encode(user); err != nil { if err := json.NewEncoder(w).Encode(user); err != nil {
h.log.Errorf("err: %+v", err) h.log.Errorf("err: %+v", err)
http.Error(w, err.Error(), http.StatusInternalServerError) httpError(w, err)
return return
} }
} }
@ -150,7 +150,7 @@ func (h *CreateUserHandler) ServeHTTP(w http.ResponseWriter, r *http.Request) {
if err := json.NewEncoder(w).Encode(user); err != nil { if err := json.NewEncoder(w).Encode(user); err != nil {
h.log.Errorf("err: %+v", err) h.log.Errorf("err: %+v", err)
http.Error(w, err.Error(), http.StatusInternalServerError) httpError(w, err)
return return
} }
} }
@ -234,7 +234,7 @@ func (h *UsersHandler) ServeHTTP(w http.ResponseWriter, r *http.Request) {
h.log.Infof("user: %s", util.Dump(user)) h.log.Infof("user: %s", util.Dump(user))
if err != nil { if err != nil {
h.log.Errorf("err: %+v", err) h.log.Errorf("err: %+v", err)
http.Error(w, err.Error(), http.StatusInternalServerError) httpError(w, err)
return return
} }
if user == nil { if user == nil {
@ -253,7 +253,7 @@ func (h *UsersHandler) ServeHTTP(w http.ResponseWriter, r *http.Request) {
h.log.Infof("user: %s", util.Dump(user)) h.log.Infof("user: %s", util.Dump(user))
if err != nil { if err != nil {
h.log.Errorf("err: %+v", err) h.log.Errorf("err: %+v", err)
http.Error(w, err.Error(), http.StatusInternalServerError) httpError(w, err)
return return
} }
if user == nil { if user == nil {
@ -273,7 +273,7 @@ func (h *UsersHandler) ServeHTTP(w http.ResponseWriter, r *http.Request) {
h.log.Infof("user: %s", util.Dump(user)) h.log.Infof("user: %s", util.Dump(user))
if err != nil { if err != nil {
h.log.Errorf("err: %+v", err) h.log.Errorf("err: %+v", err)
http.Error(w, err.Error(), http.StatusInternalServerError) httpError(w, err)
return return
} }
if user == nil { if user == nil {
@ -290,14 +290,14 @@ func (h *UsersHandler) ServeHTTP(w http.ResponseWriter, r *http.Request) {
}) })
if err != nil { if err != nil {
h.log.Errorf("err: %+v", err) h.log.Errorf("err: %+v", err)
http.Error(w, err.Error(), http.StatusInternalServerError) httpError(w, err)
return return
} }
} }
if err := json.NewEncoder(w).Encode(users); err != nil { if err := json.NewEncoder(w).Encode(users); err != nil {
h.log.Errorf("err: %+v", err) h.log.Errorf("err: %+v", err)
http.Error(w, err.Error(), http.StatusInternalServerError) httpError(w, err)
return return
} }
} }
@ -349,7 +349,7 @@ func (h *CreateUserLAHandler) ServeHTTP(w http.ResponseWriter, r *http.Request)
if err := json.NewEncoder(w).Encode(user); err != nil { if err := json.NewEncoder(w).Encode(user); err != nil {
h.log.Errorf("err: %+v", err) h.log.Errorf("err: %+v", err)
http.Error(w, err.Error(), http.StatusInternalServerError) httpError(w, err)
return return
} }
} }
@ -422,7 +422,7 @@ func (h *UpdateUserLAHandler) ServeHTTP(w http.ResponseWriter, r *http.Request)
if err := json.NewEncoder(w).Encode(user); err != nil { if err := json.NewEncoder(w).Encode(user); err != nil {
h.log.Errorf("err: %+v", err) h.log.Errorf("err: %+v", err)
http.Error(w, err.Error(), http.StatusInternalServerError) httpError(w, err)
return return
} }
} }
@ -467,7 +467,7 @@ func (h *CreateUserTokenHandler) ServeHTTP(w http.ResponseWriter, r *http.Reques
} }
if err := json.NewEncoder(w).Encode(resp); err != nil { if err := json.NewEncoder(w).Encode(resp); err != nil {
h.log.Errorf("err: %+v", err) h.log.Errorf("err: %+v", err)
http.Error(w, err.Error(), http.StatusInternalServerError) httpError(w, err)
return return
} }
} }

9
internal/services/configstore/api/variable.go
View File

@ -70,12 +70,14 @@ func (h *VariablesHandler) ServeHTTP(w http.ResponseWriter, r *http.Request) {
return err return err
}) })
if err != nil { if err != nil {
http.Error(w, err.Error(), http.StatusInternalServerError) h.log.Errorf("err: %+v", err)
httpError(w, err)
return return
} }
if err := json.NewEncoder(w).Encode(variables); err != nil { if err := json.NewEncoder(w).Encode(variables); err != nil {
http.Error(w, err.Error(), http.StatusInternalServerError) h.log.Errorf("err: %+v", err)
httpError(w, err)
return return
} }
} }
@ -117,7 +119,8 @@ func (h *CreateVariableHandler) ServeHTTP(w http.ResponseWriter, r *http.Request
} }
if err := json.NewEncoder(w).Encode(variable); err != nil { if err := json.NewEncoder(w).Encode(variable); err != nil {
http.Error(w, err.Error(), http.StatusInternalServerError) h.log.Errorf("err: %+v", err)
httpError(w, err)
return return
} }
} }

3
internal/services/gateway/api/oauth2.go
View File

@ -83,7 +83,8 @@ func (h *OAuth2CallbackHandler) ServeHTTP(w http.ResponseWriter, r *http.Request
Response: response, Response: response,
} }
if err := json.NewEncoder(w).Encode(resp); err != nil { if err := json.NewEncoder(w).Encode(resp); err != nil {
http.Error(w, err.Error(), http.StatusInternalServerError) h.log.Errorf("err: %+v", err)
httpError(w, err)
return return
} }
} }

30
internal/services/gateway/api/org.go
View File

@ -60,7 +60,8 @@ func (h *CreateOrgHandler) ServeHTTP(w http.ResponseWriter, r *http.Request) {
} }
if err := json.NewEncoder(w).Encode(org); err != nil { if err := json.NewEncoder(w).Encode(org); err != nil {
http.Error(w, err.Error(), http.StatusInternalServerError) h.log.Errorf("err: %+v", err)
httpError(w, err)
return return
} }
@ -106,7 +107,8 @@ func (h *DeleteOrgHandler) ServeHTTP(w http.ResponseWriter, r *http.Request) {
http.Error(w, err.Error(), http.StatusNotFound) http.Error(w, err.Error(), http.StatusNotFound)
return return
} }
http.Error(w, err.Error(), http.StatusInternalServerError) h.log.Errorf("err: %+v", err)
httpError(w, err)
return return
} }
} }
@ -136,13 +138,15 @@ func (h *CurrentOrgHandler) ServeHTTP(w http.ResponseWriter, r *http.Request) {
http.Error(w, err.Error(), http.StatusNotFound) http.Error(w, err.Error(), http.StatusNotFound)
return return
} }
http.Error(w, err.Error(), http.StatusInternalServerError) h.log.Errorf("err: %+v", err)
httpError(w, err)
return return
} }
res := createOrgResponse(org) res := createOrgResponse(org)
if err := json.NewEncoder(w).Encode(res); err != nil { if err := json.NewEncoder(w).Encode(res); err != nil {
http.Error(w, err.Error(), http.StatusInternalServerError) h.log.Errorf("err: %+v", err)
httpError(w, err)
return return
} }
} }
@ -167,13 +171,15 @@ func (h *OrgHandler) ServeHTTP(w http.ResponseWriter, r *http.Request) {
http.Error(w, err.Error(), http.StatusNotFound) http.Error(w, err.Error(), http.StatusNotFound)
return return
} }
http.Error(w, err.Error(), http.StatusInternalServerError) h.log.Errorf("err: %+v", err)
httpError(w, err)
return return
} }
res := createOrgResponse(org) res := createOrgResponse(org)
if err := json.NewEncoder(w).Encode(res); err != nil { if err := json.NewEncoder(w).Encode(res); err != nil {
http.Error(w, err.Error(), http.StatusInternalServerError) h.log.Errorf("err: %+v", err)
httpError(w, err)
return return
} }
} }
@ -198,13 +204,15 @@ func (h *OrgByNameHandler) ServeHTTP(w http.ResponseWriter, r *http.Request) {
http.Error(w, err.Error(), http.StatusNotFound) http.Error(w, err.Error(), http.StatusNotFound)
return return
} }
http.Error(w, err.Error(), http.StatusInternalServerError) h.log.Errorf("err: %+v", err)
httpError(w, err)
return return
} }
res := createOrgResponse(org) res := createOrgResponse(org)
if err := json.NewEncoder(w).Encode(res); err != nil { if err := json.NewEncoder(w).Encode(res); err != nil {
http.Error(w, err.Error(), http.StatusInternalServerError) h.log.Errorf("err: %+v", err)
httpError(w, err)
return return
} }
} }
@ -270,7 +278,8 @@ func (h *OrgsHandler) ServeHTTP(w http.ResponseWriter, r *http.Request) {
http.Error(w, err.Error(), http.StatusNotFound) http.Error(w, err.Error(), http.StatusNotFound)
return return
} }
http.Error(w, err.Error(), http.StatusInternalServerError) h.log.Errorf("err: %+v", err)
httpError(w, err)
return return
} }
@ -283,7 +292,8 @@ func (h *OrgsHandler) ServeHTTP(w http.ResponseWriter, r *http.Request) {
} }
if err := json.NewEncoder(w).Encode(orgsResponse); err != nil { if err := json.NewEncoder(w).Encode(orgsResponse); err != nil {
http.Error(w, err.Error(), http.StatusInternalServerError) h.log.Errorf("err: %+v", err)
httpError(w, err)
return return
} }
} }

12
internal/services/gateway/api/project.go
View File

@ -84,7 +84,7 @@ func (h *CreateProjectHandler) ServeHTTP(w http.ResponseWriter, r *http.Request)
res := createProjectResponse(project) res := createProjectResponse(project)
if err := json.NewEncoder(w).Encode(res); err != nil { if err := json.NewEncoder(w).Encode(res); err != nil {
h.log.Errorf("err: %+v", err) h.log.Errorf("err: %+v", err)
http.Error(w, err.Error(), http.StatusInternalServerError) httpError(w, err)
return return
} }
} }
@ -111,7 +111,7 @@ func (h *ProjectReconfigHandler) ServeHTTP(w http.ResponseWriter, r *http.Reques
if err := h.ch.ReconfigProject(ctx, projectID); err != nil { if err := h.ch.ReconfigProject(ctx, projectID); err != nil {
h.log.Errorf("err: %+v", err) h.log.Errorf("err: %+v", err)
http.Error(w, err.Error(), http.StatusInternalServerError) httpError(w, err)
return return
} }
} }
@ -141,7 +141,7 @@ func (h *DeleteProjectHandler) ServeHTTP(w http.ResponseWriter, r *http.Request)
return return
} }
h.log.Errorf("err: %+v", err) h.log.Errorf("err: %+v", err)
http.Error(w, err.Error(), http.StatusInternalServerError) httpError(w, err)
return return
} }
@ -152,7 +152,7 @@ func (h *DeleteProjectHandler) ServeHTTP(w http.ResponseWriter, r *http.Request)
return return
} }
h.log.Errorf("err: %+v", err) h.log.Errorf("err: %+v", err)
http.Error(w, err.Error(), http.StatusInternalServerError) httpError(w, err)
return return
} }
} }
@ -182,14 +182,14 @@ func (h *ProjectHandler) ServeHTTP(w http.ResponseWriter, r *http.Request) {
return return
} }
h.log.Errorf("err: %+v", err) h.log.Errorf("err: %+v", err)
http.Error(w, err.Error(), http.StatusInternalServerError) httpError(w, err)
return return
} }
res := createProjectResponse(project) res := createProjectResponse(project)
if err := json.NewEncoder(w).Encode(res); err != nil { if err := json.NewEncoder(w).Encode(res); err != nil {
h.log.Errorf("err: %+v", err) h.log.Errorf("err: %+v", err)
http.Error(w, err.Error(), http.StatusInternalServerError) httpError(w, err)
return return
} }
} }

14
internal/services/gateway/api/projectgroup.go
View File

@ -76,7 +76,7 @@ func (h *CreateProjectGroupHandler) ServeHTTP(w http.ResponseWriter, r *http.Req
if err := json.NewEncoder(w).Encode(projectGroup); err != nil { if err := json.NewEncoder(w).Encode(projectGroup); err != nil {
h.log.Errorf("err: %+v", err) h.log.Errorf("err: %+v", err)
http.Error(w, err.Error(), http.StatusInternalServerError) httpError(w, err)
return return
} }
} }
@ -107,14 +107,14 @@ func (h *ProjectGroupHandler) ServeHTTP(w http.ResponseWriter, r *http.Request)
return return
} }
h.log.Errorf("err: %+v", err) h.log.Errorf("err: %+v", err)
http.Error(w, err.Error(), http.StatusInternalServerError) httpError(w, err)
return return
} }
res := createProjectGroupResponse(projectGroup) res := createProjectGroupResponse(projectGroup)
if err := json.NewEncoder(w).Encode(res); err != nil { if err := json.NewEncoder(w).Encode(res); err != nil {
h.log.Errorf("err: %+v", err) h.log.Errorf("err: %+v", err)
http.Error(w, err.Error(), http.StatusInternalServerError) httpError(w, err)
return return
} }
} }
@ -145,7 +145,7 @@ func (h *ProjectGroupProjectsHandler) ServeHTTP(w http.ResponseWriter, r *http.R
return return
} }
h.log.Errorf("err: %+v", err) h.log.Errorf("err: %+v", err)
http.Error(w, err.Error(), http.StatusInternalServerError) httpError(w, err)
return return
} }
@ -156,7 +156,7 @@ func (h *ProjectGroupProjectsHandler) ServeHTTP(w http.ResponseWriter, r *http.R
if err := json.NewEncoder(w).Encode(projects); err != nil { if err := json.NewEncoder(w).Encode(projects); err != nil {
h.log.Errorf("err: %+v", err) h.log.Errorf("err: %+v", err)
http.Error(w, err.Error(), http.StatusInternalServerError) httpError(w, err)
return return
} }
} }
@ -187,7 +187,7 @@ func (h *ProjectGroupSubgroupsHandler) ServeHTTP(w http.ResponseWriter, r *http.
return return
} }
h.log.Errorf("err: %+v", err) h.log.Errorf("err: %+v", err)
http.Error(w, err.Error(), http.StatusInternalServerError) httpError(w, err)
return return
} }
@ -198,7 +198,7 @@ func (h *ProjectGroupSubgroupsHandler) ServeHTTP(w http.ResponseWriter, r *http.
if err := json.NewEncoder(w).Encode(subgroups); err != nil { if err := json.NewEncoder(w).Encode(subgroups); err != nil {
h.log.Errorf("err: %+v", err) h.log.Errorf("err: %+v", err)
http.Error(w, err.Error(), http.StatusInternalServerError) httpError(w, err)
return return
} }
} }

15
internal/services/gateway/api/remotesource.go
View File

@ -66,7 +66,8 @@ func (h *CreateRemoteSourceHandler) ServeHTTP(w http.ResponseWriter, r *http.Req
res := createRemoteSourceResponse(rs) res := createRemoteSourceResponse(rs)
if err := json.NewEncoder(w).Encode(res); err != nil { if err := json.NewEncoder(w).Encode(res); err != nil {
http.Error(w, err.Error(), http.StatusInternalServerError) h.log.Errorf("err: %+v", err)
httpError(w, err)
return return
} }
@ -158,13 +159,15 @@ func (h *RemoteSourceHandler) ServeHTTP(w http.ResponseWriter, r *http.Request)
http.Error(w, err.Error(), http.StatusNotFound) http.Error(w, err.Error(), http.StatusNotFound)
return return
} }
http.Error(w, err.Error(), http.StatusInternalServerError) h.log.Errorf("err: %+v", err)
httpError(w, err)
return return
} }
res := createRemoteSourceResponse(rs) res := createRemoteSourceResponse(rs)
if err := json.NewEncoder(w).Encode(res); err != nil { if err := json.NewEncoder(w).Encode(res); err != nil {
http.Error(w, err.Error(), http.StatusInternalServerError) h.log.Errorf("err: %+v", err)
httpError(w, err)
return return
} }
} }
@ -213,7 +216,8 @@ func (h *RemoteSourcesHandler) ServeHTTP(w http.ResponseWriter, r *http.Request)
http.Error(w, err.Error(), http.StatusNotFound) http.Error(w, err.Error(), http.StatusNotFound)
return return
} }
http.Error(w, err.Error(), http.StatusInternalServerError) h.log.Errorf("err: %+v", err)
httpError(w, err)
return return
} }
@ -223,7 +227,8 @@ func (h *RemoteSourcesHandler) ServeHTTP(w http.ResponseWriter, r *http.Request)
} }
if err := json.NewEncoder(w).Encode(remoteSources); err != nil { if err := json.NewEncoder(w).Encode(remoteSources); err != nil {
http.Error(w, err.Error(), http.StatusInternalServerError) h.log.Errorf("err: %+v", err)
httpError(w, err)
return return
} }
} }

9
internal/services/gateway/api/repos.go
View File

@ -42,7 +42,8 @@ func (h *ReposHandler) ServeHTTP(w http.ResponseWriter, r *http.Request) {
u, err := url.Parse(h.gitServerURL) u, err := url.Parse(h.gitServerURL)
if err != nil { if err != nil {
http.Error(w, err.Error(), http.StatusInternalServerError) h.log.Errorf("err: %+v", err)
httpError(w, err)
return return
} }
u.Path = path u.Path = path
@ -69,7 +70,8 @@ func (h *ReposHandler) ServeHTTP(w http.ResponseWriter, r *http.Request) {
resp, err := http.DefaultClient.Do(req) resp, err := http.DefaultClient.Do(req)
if err != nil { if err != nil {
http.Error(w, err.Error(), http.StatusInternalServerError) h.log.Errorf("err: %+v", err)
httpError(w, err)
return return
} }
@ -85,7 +87,8 @@ func (h *ReposHandler) ServeHTTP(w http.ResponseWriter, r *http.Request) {
defer resp.Body.Close() defer resp.Body.Close()
// copy response body // copy response body
if _, err := io.Copy(w, resp.Body); err != nil { if _, err := io.Copy(w, resp.Body); err != nil {
http.Error(w, err.Error(), http.StatusInternalServerError) h.log.Errorf("err: %+v", err)
httpError(w, err)
return return
} }
} }

27
internal/services/gateway/api/run.go
View File

@ -207,13 +207,15 @@ func (h *RunHandler) ServeHTTP(w http.ResponseWriter, r *http.Request) {
http.Error(w, err.Error(), http.StatusNotFound) http.Error(w, err.Error(), http.StatusNotFound)
return return
} }
http.Error(w, err.Error(), http.StatusInternalServerError) h.log.Errorf("err: %+v", err)
httpError(w, err)
return return
} }
res := createRunResponse(runResp.Run, runResp.RunConfig) res := createRunResponse(runResp.Run, runResp.RunConfig)
if err := json.NewEncoder(w).Encode(res); err != nil { if err := json.NewEncoder(w).Encode(res); err != nil {
http.Error(w, err.Error(), http.StatusInternalServerError) h.log.Errorf("err: %+v", err)
httpError(w, err)
return return
} }
} }
@ -239,7 +241,8 @@ func (h *RuntaskHandler) ServeHTTP(w http.ResponseWriter, r *http.Request) {
http.Error(w, err.Error(), http.StatusNotFound) http.Error(w, err.Error(), http.StatusNotFound)
return return
} }
http.Error(w, err.Error(), http.StatusInternalServerError) h.log.Errorf("err: %+v", err)
httpError(w, err)
return return
} }
@ -256,7 +259,8 @@ func (h *RuntaskHandler) ServeHTTP(w http.ResponseWriter, r *http.Request) {
res := createRunTaskResponse(rt, rct) res := createRunTaskResponse(rt, rct)
if err := json.NewEncoder(w).Encode(res); err != nil { if err := json.NewEncoder(w).Encode(res); err != nil {
http.Error(w, err.Error(), http.StatusInternalServerError) h.log.Errorf("err: %+v", err)
httpError(w, err)
return return
} }
} }
@ -340,7 +344,8 @@ func (h *RunsHandler) ServeHTTP(w http.ResponseWriter, r *http.Request) {
http.Error(w, err.Error(), http.StatusNotFound) http.Error(w, err.Error(), http.StatusNotFound)
return return
} }
http.Error(w, err.Error(), http.StatusInternalServerError) h.log.Errorf("err: %+v", err)
httpError(w, err)
return return
} }
@ -350,7 +355,8 @@ func (h *RunsHandler) ServeHTTP(w http.ResponseWriter, r *http.Request) {
} }
if err := json.NewEncoder(w).Encode(runs); err != nil { if err := json.NewEncoder(w).Encode(runs); err != nil {
http.Error(w, err.Error(), http.StatusInternalServerError) h.log.Errorf("err: %+v", err)
httpError(w, err)
return return
} }
} }
@ -403,7 +409,8 @@ func (h *RunActionsHandler) ServeHTTP(w http.ResponseWriter, r *http.Request) {
http.Error(w, err.Error(), http.StatusNotFound) http.Error(w, err.Error(), http.StatusNotFound)
return return
} }
http.Error(w, err.Error(), http.StatusInternalServerError) h.log.Errorf("err: %+v", err)
httpError(w, err)
return return
} }
@ -418,7 +425,8 @@ func (h *RunActionsHandler) ServeHTTP(w http.ResponseWriter, r *http.Request) {
http.Error(w, err.Error(), http.StatusNotFound) http.Error(w, err.Error(), http.StatusNotFound)
return return
} }
http.Error(w, err.Error(), http.StatusInternalServerError) h.log.Errorf("err: %+v", err)
httpError(w, err)
return return
} }
} }
@ -488,7 +496,8 @@ func (h *LogsHandler) ServeHTTP(w http.ResponseWriter, r *http.Request) {
http.Error(w, err.Error(), http.StatusNotFound) http.Error(w, err.Error(), http.StatusNotFound)
return return
} }
http.Error(w, "", http.StatusInternalServerError) h.log.Errorf("err: %+v", err)
httpError(w, err)
return return
} }

9
internal/services/gateway/api/secret.go
View File

@ -79,7 +79,8 @@ func (h *SecretHandler) ServeHTTP(w http.ResponseWriter, r *http.Request) {
} }
if err := json.NewEncoder(w).Encode(secrets); err != nil { if err := json.NewEncoder(w).Encode(secrets); err != nil {
http.Error(w, err.Error(), http.StatusInternalServerError) h.log.Errorf("err: %+v", err)
httpError(w, err)
return return
} }
} }
@ -149,7 +150,8 @@ func (h *CreateSecretHandler) ServeHTTP(w http.ResponseWriter, r *http.Request)
res := createSecretResponse(s) res := createSecretResponse(s)
if err := json.NewEncoder(w).Encode(res); err != nil { if err := json.NewEncoder(w).Encode(res); err != nil {
http.Error(w, err.Error(), http.StatusInternalServerError) h.log.Errorf("err: %+v", err)
httpError(w, err)
return return
} }
} }
@ -187,7 +189,8 @@ func (h *DeleteSecretHandler) ServeHTTP(w http.ResponseWriter, r *http.Request)
http.Error(w, err.Error(), http.StatusNotFound) http.Error(w, err.Error(), http.StatusNotFound)
return return
} }
http.Error(w, err.Error(), http.StatusInternalServerError) h.log.Errorf("err: %+v", err)
httpError(w, err)
return return
} }
} }

52
internal/services/gateway/api/user.go
View File

@ -66,7 +66,8 @@ func (h *CreateUserHandler) ServeHTTP(w http.ResponseWriter, r *http.Request) {
res := createUserResponse(u) res := createUserResponse(u)
if err := json.NewEncoder(w).Encode(res); err != nil { if err := json.NewEncoder(w).Encode(res); err != nil {
http.Error(w, err.Error(), http.StatusInternalServerError) h.log.Errorf("err: %+v", err)
httpError(w, err)
return return
} }
} }
@ -91,7 +92,8 @@ func (h *DeleteUserHandler) ServeHTTP(w http.ResponseWriter, r *http.Request) {
http.Error(w, err.Error(), http.StatusNotFound) http.Error(w, err.Error(), http.StatusNotFound)
return return
} }
http.Error(w, err.Error(), http.StatusInternalServerError) h.log.Errorf("err: %+v", err)
httpError(w, err)
return return
} }
} }
@ -121,13 +123,15 @@ func (h *CurrentUserHandler) ServeHTTP(w http.ResponseWriter, r *http.Request) {
http.Error(w, err.Error(), http.StatusNotFound) http.Error(w, err.Error(), http.StatusNotFound)
return return
} }
http.Error(w, err.Error(), http.StatusInternalServerError) h.log.Errorf("err: %+v", err)
httpError(w, err)
return return
} }
res := createUserResponse(user) res := createUserResponse(user)
if err := json.NewEncoder(w).Encode(res); err != nil { if err := json.NewEncoder(w).Encode(res); err != nil {
http.Error(w, err.Error(), http.StatusInternalServerError) h.log.Errorf("err: %+v", err)
httpError(w, err)
return return
} }
} }
@ -152,13 +156,15 @@ func (h *UserHandler) ServeHTTP(w http.ResponseWriter, r *http.Request) {
http.Error(w, err.Error(), http.StatusNotFound) http.Error(w, err.Error(), http.StatusNotFound)
return return
} }
http.Error(w, err.Error(), http.StatusInternalServerError) h.log.Errorf("err: %+v", err)
httpError(w, err)
return return
} }
res := createUserResponse(user) res := createUserResponse(user)
if err := json.NewEncoder(w).Encode(res); err != nil { if err := json.NewEncoder(w).Encode(res); err != nil {
http.Error(w, err.Error(), http.StatusInternalServerError) h.log.Errorf("err: %+v", err)
httpError(w, err)
return return
} }
} }
@ -183,13 +189,15 @@ func (h *UserByNameHandler) ServeHTTP(w http.ResponseWriter, r *http.Request) {
http.Error(w, err.Error(), http.StatusNotFound) http.Error(w, err.Error(), http.StatusNotFound)
return return
} }
http.Error(w, err.Error(), http.StatusInternalServerError) h.log.Errorf("err: %+v", err)
httpError(w, err)
return return
} }
res := createUserResponse(user) res := createUserResponse(user)
if err := json.NewEncoder(w).Encode(res); err != nil { if err := json.NewEncoder(w).Encode(res); err != nil {
http.Error(w, err.Error(), http.StatusInternalServerError) h.log.Errorf("err: %+v", err)
httpError(w, err)
return return
} }
} }
@ -258,7 +266,8 @@ func (h *UsersHandler) ServeHTTP(w http.ResponseWriter, r *http.Request) {
http.Error(w, err.Error(), http.StatusNotFound) http.Error(w, err.Error(), http.StatusNotFound)
return return
} }
http.Error(w, err.Error(), http.StatusInternalServerError) h.log.Errorf("err: %+v", err)
httpError(w, err)
return return
} }
@ -268,7 +277,8 @@ func (h *UsersHandler) ServeHTTP(w http.ResponseWriter, r *http.Request) {
} }
if err := json.NewEncoder(w).Encode(users); err != nil { if err := json.NewEncoder(w).Encode(users); err != nil {
http.Error(w, err.Error(), http.StatusInternalServerError) h.log.Errorf("err: %+v", err)
httpError(w, err)
return return
} }
} }
@ -313,7 +323,8 @@ func (h *CreateUserLAHandler) ServeHTTP(w http.ResponseWriter, r *http.Request)
} }
if err := json.NewEncoder(w).Encode(resp); err != nil { if err := json.NewEncoder(w).Encode(resp); err != nil {
http.Error(w, err.Error(), http.StatusInternalServerError) h.log.Errorf("err: %+v", err)
httpError(w, err)
return return
} }
@ -361,7 +372,8 @@ func (h *DeleteUserLAHandler) ServeHTTP(w http.ResponseWriter, r *http.Request)
_, err := h.configstoreClient.DeleteUserLA(ctx, userName, laID) _, err := h.configstoreClient.DeleteUserLA(ctx, userName, laID)
if err != nil { if err != nil {
http.Error(w, err.Error(), http.StatusInternalServerError) h.log.Errorf("err: %+v", err)
httpError(w, err)
return return
} }
} }
@ -410,7 +422,8 @@ func (h *CreateUserTokenHandler) ServeHTTP(w http.ResponseWriter, r *http.Reques
Token: token, Token: token,
} }
if err := json.NewEncoder(w).Encode(resp); err != nil { if err := json.NewEncoder(w).Encode(resp); err != nil {
http.Error(w, err.Error(), http.StatusInternalServerError) h.log.Errorf("err: %+v", err)
httpError(w, err)
return return
} }
} }
@ -433,7 +446,8 @@ func (h *DeleteUserTokenHandler) ServeHTTP(w http.ResponseWriter, r *http.Reques
h.log.Infof("deleting user %q token %q", userName, tokenName) h.log.Infof("deleting user %q token %q", userName, tokenName)
_, err := h.configstoreClient.DeleteUserToken(ctx, userName, tokenName) _, err := h.configstoreClient.DeleteUserToken(ctx, userName, tokenName)
if err != nil { if err != nil {
http.Error(w, err.Error(), http.StatusInternalServerError) h.log.Errorf("err: %+v", err)
httpError(w, err)
return return
} }
} }
@ -474,10 +488,10 @@ func (h *RegisterUserHandler) ServeHTTP(w http.ResponseWriter, r *http.Request)
} }
if err := json.NewEncoder(w).Encode(resp); err != nil { if err := json.NewEncoder(w).Encode(resp); err != nil {
http.Error(w, err.Error(), http.StatusInternalServerError) h.log.Errorf("err: %+v", err)
httpError(w, err)
return return
} }
} }
func (h *RegisterUserHandler) registerUser(ctx context.Context, req *RegisterUserRequest) (*RegisterUserResponse, error) { func (h *RegisterUserHandler) registerUser(ctx context.Context, req *RegisterUserRequest) (*RegisterUserResponse, error) {
@ -534,7 +548,8 @@ func (h *AuthorizeHandler) ServeHTTP(w http.ResponseWriter, r *http.Request) {
} }
if err := json.NewEncoder(w).Encode(resp); err != nil { if err := json.NewEncoder(w).Encode(resp); err != nil {
http.Error(w, err.Error(), http.StatusInternalServerError) h.log.Errorf("err: %+v", err)
httpError(w, err)
return return
} }
@ -602,7 +617,8 @@ func (h *LoginUserHandler) ServeHTTP(w http.ResponseWriter, r *http.Request) {
} }
if err := json.NewEncoder(w).Encode(resp); err != nil { if err := json.NewEncoder(w).Encode(resp); err != nil {
http.Error(w, err.Error(), http.StatusInternalServerError) h.log.Errorf("err: %+v", err)
httpError(w, err)
return return
} }

10
internal/services/gateway/api/variable.go
View File

@ -133,7 +133,8 @@ func (h *VariableHandler) ServeHTTP(w http.ResponseWriter, r *http.Request) {
} }
if err := json.NewEncoder(w).Encode(variables); err != nil { if err := json.NewEncoder(w).Encode(variables); err != nil {
http.Error(w, err.Error(), http.StatusInternalServerError) h.log.Errorf("err: %+v", err)
httpError(w, err)
return return
} }
} }
@ -219,7 +220,8 @@ func (h *CreateVariableHandler) ServeHTTP(w http.ResponseWriter, r *http.Request
res := createVariableResponse(v, cssecrets) res := createVariableResponse(v, cssecrets)
if err := json.NewEncoder(w).Encode(res); err != nil { if err := json.NewEncoder(w).Encode(res); err != nil {
http.Error(w, err.Error(), http.StatusInternalServerError) h.log.Errorf("err: %+v", err)
httpError(w, err)
return return
} }
} }
@ -240,6 +242,7 @@ func (h *DeleteVariableHandler) ServeHTTP(w http.ResponseWriter, r *http.Request
parentType, parentRef, err := GetConfigTypeRef(r) parentType, parentRef, err := GetConfigTypeRef(r)
if httpError(w, err) { if httpError(w, err) {
h.log.Errorf("err: %+v", err)
return return
} }
@ -257,7 +260,8 @@ func (h *DeleteVariableHandler) ServeHTTP(w http.ResponseWriter, r *http.Request
http.Error(w, err.Error(), http.StatusNotFound) http.Error(w, err.Error(), http.StatusNotFound)
return return
} }
http.Error(w, err.Error(), http.StatusInternalServerError) h.log.Errorf("err: %+v", err)
httpError(w, err)
return return
} }
} }