use acceptedResponseStatusCode everywhere in HTTP logger (#13755)

Harshavardhana committed 4 years ago

Commit fee3f881 parent 932500e2 files · +6 −3Browse files

Unified · Split

2 files

buildscripts/minio-upgrade.sh +5 −2
internal/logger/target/http/http.go +1 −1

buildscripts/minio-upgrade.sh+5−2

@@ -37,8 +37,11 @@ verify_checksum_mc() { · 36 unmodified lines
37	37	}
38	38
39	39	add_alias() {
40		− until (mc alias set minio http://127.0.0.1:9000 minioadmin minioadmin); do
41		− echo "...waiting... for 5secs" && sleep 5
	40	+ for i in $(seq 1 4); do
	41	+ echo "... attempting to add alias $i"
	42	+ until (mc alias set minio http://127.0.0.1:9000 minioadmin minioadmin); do
	43	+ echo "...waiting... for 5secs" && sleep 5
	44	+ done
42	45	done
43	46	}
44	47

internal/logger/target/http/http.go+1−1

@@ -157,7 +157,7 @@ func (h *Target) startHTTPLogger() { · 156 unmodified lines
157	157	// Drain any response.
158	158	xhttp.DrainBody(resp.Body)
159	159
160		− if resp.StatusCode != http.StatusOK {
	160	+ if !acceptedResponseStatusCode(resp.StatusCode) {
161	161	switch resp.StatusCode {
162	162	case http.StatusForbidden:
163	163	h.config.LogOnce(ctx, fmt.Errorf("%s returned '%s', please check if your auth token is correctly set", h.config.Endpoint, resp.Status), h.config.Endpoint)