[ARVADOS] updated: 1.3.0-1798-g857533188

[Git user]

Summary of changes:
 sdk/go/httpserver/logger.go         |  8 +++--
 sdk/go/httpserver/logger_test.go    | 68 ++++++++++++++++++++++++++++++++-----
 sdk/go/httpserver/responsewriter.go | 20 +++++++++++
 3 files changed, 85 insertions(+), 11 deletions(-)

       via  857533188539ebbcb9cf3874342784b811251b3a (commit)
       via  e701875382821088123769823f6b66a5a4ae2907 (commit)
       via  7db3ceda16742b65d73ebbc05d02351a5e0496bd (commit)
      from  4cadc4832ed4dd58e841628232c472e99b6f7d1b (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 857533188539ebbcb9cf3874342784b811251b3a
Merge: 4cadc4832 e70187538
Author: Tom Clegg <tclegg at veritasgenetics.com>
Date:   Thu Oct 24 15:31:02 2019 -0400

    Merge branch '15713-controller-error-log'
    
    refs #15713
    
    Arvados-DCO-1.1-Signed-off-by: Tom Clegg <tclegg at veritasgenetics.com>


commit e701875382821088123769823f6b66a5a4ae2907
Author: Tom Clegg <tclegg at veritasgenetics.com>
Date:   Thu Oct 24 15:30:13 2019 -0400

    15713: Don't sniff non-error response body.
    
    Arvados-DCO-1.1-Signed-off-by: Tom Clegg <tclegg at veritasgenetics.com>

diff --git a/sdk/go/httpserver/responsewriter.go b/sdk/go/httpserver/responsewriter.go
index 26aee65c5..884f0d8d7 100644
--- a/sdk/go/httpserver/responsewriter.go
+++ b/sdk/go/httpserver/responsewriter.go
@@ -47,8 +47,9 @@ func (w *responseWriter) WriteHeader(s int) {
 func (w *responseWriter) Write(data []byte) (n int, err error) {
 	if w.wroteStatus == 0 {
 		w.WriteHeader(http.StatusOK)
+	} else if w.wroteStatus >= 400 {
+		w.sniff(data)
 	}
-	w.sniff(data)
 	n, err = w.ResponseWriter.Write(data)
 	w.wroteBodyBytes += n
 	w.err = err

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


hooks/post-receive
--