scottk/vault
1
0
Fork 0
mirror of https://github.com/optim-enterprises-bv/vault.git synced 2025-11-02 03:27:54 +00:00
Code Issues Packages Projects Releases Wiki Activity

[VAULT-2825] Fix erroneous 500 resp for field validation errors (#12042)

Browse Source 
* [VAULT-2825] Correctly respond with 400 rather than 500 for field validation errors

* [VAULT-2825] Add changelog entry

* [VAULT-2825] Simplify test assertion
This commit is contained in:
Chris Capurso
2021-07-12 13:39:28 -04:00
committed by GitHub
parent c286fcc433
commit e11eb36bc6
3 changed files with 10 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
sdk/framework/backend.go
View File

@@ -267,7 +267,7 @@ func (b *Backend) HandleRequest(ctx context.Context, req *logical.Request) (*log
if req.Operation != logical.HelpOperation {
err := fd.Validate()
if err != nil {
return nil, err
return logical.ErrorResponse(fmt.Sprintf("Field validation failed: %s", err.Error())), nil
}
}

9
sdk/framework/backend_test.go
View File

@@ -240,14 +240,17 @@ func TestBackendHandleRequest_badwrite(t *testing.T) {
},
}
_, err := b.HandleRequest(context.Background(), &logical.Request{
resp, err := b.HandleRequest(context.Background(), &logical.Request{
Operation: logical.UpdateOperation,
Path: "foo/bar",
Data: map[string]interface{}{"value": "3false3"},
})
if err != nil {
t.Fatalf("err: %s", err)
}
if err == nil {
t.Fatalf("should have thrown a conversion error")
if !strings.Contains(resp.Data["error"].(string), "Field validation failed") {
t.Fatalf("bad: %#v", resp)
}
}