From ba1989fe848758e656680f5dc08ee6c9307fa629 Mon Sep 17 00:00:00 2001 From: Harshavardhana Date: Wed, 22 Sep 2021 00:46:07 -0700 Subject: [PATCH] do not panic if DNS_WEBHOOK_ENDPOINT is not reachable --- cmd/config/dns/operator_dns.go | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/cmd/config/dns/operator_dns.go b/cmd/config/dns/operator_dns.go index e065a32fc..a30e926dc 100644 --- a/cmd/config/dns/operator_dns.go +++ b/cmd/config/dns/operator_dns.go @@ -89,16 +89,19 @@ func (c *OperatorDNS) Put(bucket string) error { if err = c.addAuthHeader(req); err != nil { return newError(bucket, err) } + resp, err := c.httpClient.Do(req) if err != nil { if derr := c.Delete(bucket); derr != nil { return newError(bucket, derr) } + return err } - var errorStringBuilder strings.Builder - io.Copy(&errorStringBuilder, io.LimitReader(resp.Body, resp.ContentLength)) - xhttp.DrainBody(resp.Body) + defer xhttp.DrainBody(resp.Body) + if resp.StatusCode != http.StatusOK { + var errorStringBuilder strings.Builder + io.Copy(&errorStringBuilder, io.LimitReader(resp.Body, resp.ContentLength)) errorString := errorStringBuilder.String() switch resp.StatusCode { case http.StatusConflict: