Commit a028cdc

committed

Merge branch '434-config-error-response' into 'master'

fix(engine): use the bad request status instead of server error (#434) See merge request postgres-ai/database-lab!609

2 parents 73db6f6 + 4e321f3 commit a028cdcCopy full SHA for a028cdc

File tree

-2

lines changed

-2

lines changed

Lines changed: 2 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -28,7 +28,7 @@ const connectionCheckTimeout = 10 * time.Second`
`28`	`28`	`func (s Server) getProjectedAdminConfig(w http.ResponseWriter, r http.Request) {`
`29`	`29`	`cfg, err := s.projectedAdminConfig()`
`30`	`30`	`if err != nil {`
`31`		`- api.SendError(w, r, err)`
	`31`	`+ api.SendBadRequestError(w, r, err.Error())`
`32`	`32`	`return`
`33`	`33`	`}`
`34`	`34`
`@@ -41,7 +41,7 @@ func (s Server) getProjectedAdminConfig(w http.ResponseWriter, r http.Request)`
`41`	`41`	`func (s Server) getAdminConfigYaml(w http.ResponseWriter, r http.Request) {`
`42`	`42`	`cfg, err := adminConfigYaml()`
`43`	`43`	`if err != nil {`
`44`		`- api.SendError(w, r, err)`
	`44`	`+ api.SendBadRequestError(w, r, err.Error())`
`45`	`45`	`return`
`46`	`46`	`}`
`47`	`47`

Comments

(0)