From 1444e34521e565d2077e684863e68cb9befdef3b Mon Sep 17 00:00:00 2001 From: Xe Date: Fri, 22 Jul 2022 12:38:24 -0400 Subject: [PATCH] cmd/gitops-pusher: fix minor bug with ACL tests Signed-off-by: Xe --- cmd/gitops-pusher/gitops-pusher.go | 15 +++++++++------ 1 file changed, 9 insertions(+), 6 deletions(-) diff --git a/cmd/gitops-pusher/gitops-pusher.go b/cmd/gitops-pusher/gitops-pusher.go index fc1861bdb..1f182cf4c 100644 --- a/cmd/gitops-pusher/gitops-pusher.go +++ b/cmd/gitops-pusher/gitops-pusher.go @@ -15,6 +15,7 @@ "fmt" "log" "net/http" + "net/http/httputil" "os" "regexp" "strings" @@ -237,12 +238,6 @@ func testNewACLs(ctx context.Context, tailnet, apiKey, policyFname string) error } defer resp.Body.Close() - got := resp.StatusCode - want := http.StatusOK - if got != want { - return fmt.Errorf("wanted HTTP status code %d but got %d", want, got) - } - var ate ACLTestError err = json.NewDecoder(resp.Body).Decode(&ate) if err != nil { @@ -253,6 +248,14 @@ func testNewACLs(ctx context.Context, tailnet, apiKey, policyFname string) error return ate } + got := resp.StatusCode + want := http.StatusOK + if got != want { + data, _ := httputil.DumpResponse(resp, true) + os.Stderr.Write(data) + return fmt.Errorf("wanted HTTP status code %d but got %d", want, got) + } + return nil }