Fix R2 create bucket endpoint (#1653)

* Fix R2 create bucket endpoint * changeset * Styling * Fix test
cloudflare · Aug 22, 2022 · 46b73b5 · 46b73b5
1 parent 61e3f00
commit 46b73b5
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 7 deletions.
diff --git a/.changeset/twelve-peaches-punch.md b/.changeset/twelve-peaches-punch.md
@@ -0,0 +1,5 @@
+---
+"wrangler": patch
+---
+
+Fixed R2 create bucket API endpoint. The `wrangler r2 bucket create` command should work again
diff --git a/packages/wrangler/src/__tests__/r2.test.ts b/packages/wrangler/src/__tests__/r2.test.ts
@@ -82,10 +82,11 @@ describe("wrangler", () => {
 				function mockCreateRequest(expectedBucketName: string) {
 					const requests = { count: 0 };
 					setMockResponse(
-						"/accounts/:accountId/r2/buckets/:bucketName",
-						"PUT",
-						([_url, accountId, bucketName]) => {
+						"/accounts/:accountId/r2/buckets",
+						"POST",
+						([_url, accountId], { body }) => {
 							expect(accountId).toEqual("some-account-id");
+							const bucketName = JSON.parse(body as string).name;
 							expect(bucketName).toEqual(expectedBucketName);
 							requests.count += 1;
 						}

diff --git a/packages/wrangler/src/r2.ts b/packages/wrangler/src/r2.ts
@@ -33,10 +33,10 @@ export async function createR2Bucket(
 	accountId: string,
 	bucketName: string
 ): Promise<void> {
-	return await fetchResult<void>(
-		`/accounts/${accountId}/r2/buckets/${bucketName}`,
-		{ method: "PUT" }
-	);
+	return await fetchResult<void>(`/accounts/${accountId}/r2/buckets`, {
+		method: "POST",
+		body: JSON.stringify({ name: bucketName }),
+	});
 }
 
 /**