From: Tom Clegg <tclegg@veritasgenetics.com>
Date: Wed, 26 Jun 2019 16:00:05 +0000 (-0400)
Subject: 14287: Don't delete non-writable attrs in create/update reqs.
X-Git-Url: https://git.arvados.org/arvados.git/commitdiff_plain/49ec8f92ef81041d58fc66593c9a33bcbe7c3d22

14287: Don't delete non-writable attrs in create/update reqs.

Arvados-DCO-1.1-Signed-off-by: Tom Clegg <tclegg@veritasgenetics.com>
---

diff --git a/lib/controller/router/request.go b/lib/controller/router/request.go
index c219b1e1de..377f7243c0 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)
 	}