[ARVADOS] updated: 1.1.4-143-g7d96319

Git user git at public.curoverse.com
Tue May 1 11:17:32 EDT 2018


Summary of changes:
 services/keepproxy/keepproxy.go    | 4 ++--
 services/keepproxy/proxy_client.go | 6 ++----
 2 files changed, 4 insertions(+), 6 deletions(-)

       via  7d96319e476c866f33f119dddd534fa6cbe482fb (commit)
      from  97ad08809663d2adcb00947a1dcc0f922f5937bc (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.


commit 7d96319e476c866f33f119dddd534fa6cbe482fb
Author: Tom Clegg <tclegg at veritasgenetics.com>
Date:   Tue May 1 11:16:47 2018 -0400

    12167: Pass request ID via keepclient instead of custom code.
    
    Arvados-DCO-1.1-Signed-off-by: Tom Clegg <tclegg at veritasgenetics.com>

diff --git a/services/keepproxy/keepproxy.go b/services/keepproxy/keepproxy.go
index c2bf6bc..7baadab 100644
--- a/services/keepproxy/keepproxy.go
+++ b/services/keepproxy/keepproxy.go
@@ -622,13 +622,13 @@ func (h *proxyHandler) Index(resp http.ResponseWriter, req *http.Request) {
 
 func (h *proxyHandler) makeKeepClient(req *http.Request) *keepclient.KeepClient {
 	kc := *h.KeepClient
+	kc.RequestID = req.Header.Get("X-Request-Id")
 	kc.HTTPClient = &proxyClient{
 		client: &http.Client{
 			Timeout:   h.timeout,
 			Transport: h.transport,
 		},
-		proto:     req.Proto,
-		requestID: req.Header.Get("X-Request-Id"),
+		proto: req.Proto,
 	}
 	return &kc
 }
diff --git a/services/keepproxy/proxy_client.go b/services/keepproxy/proxy_client.go
index 06dfa1c..0faf4ae 100644
--- a/services/keepproxy/proxy_client.go
+++ b/services/keepproxy/proxy_client.go
@@ -13,13 +13,11 @@ import (
 var viaAlias = "keepproxy"
 
 type proxyClient struct {
-	client    keepclient.HTTPClient
-	proto     string
-	requestID string
+	client keepclient.HTTPClient
+	proto  string
 }
 
 func (pc *proxyClient) Do(req *http.Request) (*http.Response, error) {
 	req.Header.Add("Via", pc.proto+" "+viaAlias)
-	req.Header.Set("X-Request-Id", pc.requestID)
 	return pc.client.Do(req)
 }

-----------------------------------------------------------------------


hooks/post-receive
-- 




More information about the arvados-commits mailing list