From 095a66fb83d1ad5d8794bc50715ea4be953336f5 Mon Sep 17 00:00:00 2001
From: Moritz Sanft <58110325+msanft@users.noreply.github.com>
Date: Fri, 7 Jun 2024 15:18:34 +0200
Subject: [PATCH] cli: return a more helpful error message on MAA patch failure
 (#3153)
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

* cli: return a more helpful error message on maa patch failure

* Update internal/maa/patch.go

Co-authored-by: Daniel Weiße <66256922+daniel-weisse@users.noreply.github.com>

---------

Co-authored-by: Daniel Weiße <66256922+daniel-weisse@users.noreply.github.com>
---
 internal/maa/patch.go | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/internal/maa/patch.go b/internal/maa/patch.go
index 5dfed9435..d9ac35b4c 100644
--- a/internal/maa/patch.go
+++ b/internal/maa/patch.go
@@ -9,6 +9,7 @@ import (
 	"context"
 	"encoding/base64"
 	"fmt"
+	"io"
 	"net/http"
 
 	"github.com/Azure/azure-sdk-for-go/profiles/latest/attestation/attestation"
@@ -55,10 +56,11 @@ func (p AzurePolicyPatcher) Patch(ctx context.Context, attestationURL string) er
 	if err != nil {
 		return fmt.Errorf("sending request: %w", err)
 	}
-	resp.Body.Close()
+	defer resp.Body.Close()
 
 	if resp.StatusCode != http.StatusOK {
-		return fmt.Errorf("updating attestation policy: unexpected status code: %s", resp.Status)
+		body, _ := io.ReadAll(resp.Body)
+		return fmt.Errorf("updating attestation policy: unexpected status code: %s: %s", resp.Status, string(body))
 	}
 
 	return nil