fix: defer body close returned as was

For some reason it was causing double notifications

Author: Michaelpalacce

config/samples/ddns_v1alpha1_provider.yaml

@@ -4,7 +4,7 @@ metadata:
   labels:
     app.kubernetes.io/name: go-ddns-controller
     app.kubernetes.io/managed-by: kustomize
-  name: provider-sample
+  name: cloudflare-provider
 spec:
   name: Cloudflare
   secretName: cloudflare

internal/network/http.go

@@ -10,11 +10,7 @@ import (
 // Will also close the ReadStream
 func GetBody(url string) ([]byte, error) {
 	if resp, err := http.Get(url); err == nil {
-		defer func() {
-			if err := resp.Body.Close(); err != nil {
-				fmt.Println("Error while closing response body")
-			}
-		}()
+		defer resp.Body.Close()
 
 		if body, err := io.ReadAll(resp.Body); err == nil {
 			return body, nil

internal/notifiers/webhook.go

@@ -62,11 +62,7 @@ func (w *WebhookNotifier) sendToWebhook(data string) error {
 		return err
 	}
 
-	defer func() {
-		if err := resp.Body.Close(); err != nil {
-			slog.Error("error while closing response body", "error", err)
-		}
-	}()
+	defer resp.Body.Close()
 
 	slog.Debug("Status Code", "code", resp.StatusCode)