[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