From 071eb9f6bf06dd7a351d87191db613b66337fef2 Mon Sep 17 00:00:00 2001 From: Pengfei Ni Date: Tue, 4 Sep 2018 14:26:19 +0800 Subject: [PATCH] Fix panic when processing http response --- pkg/cloudprovider/providers/azure/azure_backoff.go | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/pkg/cloudprovider/providers/azure/azure_backoff.go b/pkg/cloudprovider/providers/azure/azure_backoff.go index 02a08d11707..44998098033 100644 --- a/pkg/cloudprovider/providers/azure/azure_backoff.go +++ b/pkg/cloudprovider/providers/azure/azure_backoff.go @@ -361,7 +361,12 @@ func processHTTPRetryResponse(resp *http.Response, err error) (bool, error) { } if shouldRetryHTTPRequest(resp, err) { - glog.Errorf("processHTTPRetryResponse: backoff failure, will retry, HTTP response=%d, err=%v", resp.StatusCode, err) + if err != nil { + glog.Errorf("processHTTPRetryResponse: backoff failure, will retry, err=%v", err) + } else { + glog.Errorf("processHTTPRetryResponse: backoff failure, will retry, HTTP response=%d", resp.StatusCode) + } + // suppress the error object so that backoff process continues return false, nil }