Use PUT request for uploading to S3, instead of POST

wallrj · wallrj · commit 5507b54183e1 · 2025-08-05T16:44:27.000+01:00
Signed-off-by: Richard Wall &lt;richard.wall@venafi.com&gt;
diff --git a/pkg/internal/cyberark/dataupload/dataupload.go b/pkg/internal/cyberark/dataupload/dataupload.go
@@ -67,7 +67,7 @@ func (c *CyberArkClient) PostDataReadingsWithOptions(ctx context.Context, payloa
 		return fmt.Errorf("while retrieving presigned upload URL: %v", err)
 	}
 
-	req, err := http.NewRequestWithContext(ctx, http.MethodPost, presignedUploadURL, encodedBody)
+	req, err := http.NewRequestWithContext(ctx, http.MethodPut, presignedUploadURL, encodedBody)
 	if err != nil {
 		return err
 	}

Original file line number	Diff line number	Diff line change
`@@ -67,7 +67,7 @@ func (c *CyberArkClient) PostDataReadingsWithOptions(ctx context.Context, payloa`
`67`	`67`	`return fmt.Errorf("while retrieving presigned upload URL: %v", err)`
`68`	`68`	`}`
`69`	`69`
`70`		`- req, err := http.NewRequestWithContext(ctx, http.MethodPost, presignedUploadURL, encodedBody)`
	`70`	`+ req, err := http.NewRequestWithContext(ctx, http.MethodPut, presignedUploadURL, encodedBody)`
`71`	`71`	`if err != nil {`
`72`	`72`	`return err`
`73`	`73`	`}`