[ARVADOS] created: 1.3.0-1162-g3a2857440
Git user
git at public.curoverse.com
Wed Jun 26 18:42:29 UTC 2019
at 3a28574402bbeb5df3ea8f32f2f60a7a2f20e4fa (commit)
commit 3a28574402bbeb5df3ea8f32f2f60a7a2f20e4fa
Author: Tom Clegg <tclegg at veritasgenetics.com>
Date: Wed Jun 26 13:40:36 2019 -0400
14287: Don't delete non-writable attrs in create/update reqs.
This is already handled by clients and/or RailsAPI anyway.
Arvados-DCO-1.1-Signed-off-by: Tom Clegg <tclegg at veritasgenetics.com>
diff --git a/lib/controller/router/request.go b/lib/controller/router/request.go
index c219b1e1d..377f7243c 100644
--- a/lib/controller/router/request.go
+++ b/lib/controller/router/request.go
@@ -115,13 +115,6 @@ func (rtr *router) loadRequestParams(req *http.Request, attrsKey string) (map[st
}
if v, ok := params[attrsKey]; ok && attrsKey != "" {
- if v, ok := v.(map[string]interface{}); ok {
- // Delete field(s) that appear in responses
- // but not in update attrs, so clients can
- // fetch-modify-update.
- delete(v, "etag")
- delete(v, "unsigned_manifest_text")
- }
params["attrs"] = v
delete(params, attrsKey)
}
-----------------------------------------------------------------------
hooks/post-receive
--
More information about the arvados-commits
mailing list