Not including detail id in wrangler output (#6671)

cloudflare · Sep 12, 2024 · 48eeff4 · 48eeff4
1 parent 1986492
commit 48eeff4
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 4 deletions.
diff --git a/.changeset/smooth-bananas-obey.md b/.changeset/smooth-bananas-obey.md
@@ -0,0 +1,5 @@
+---
+"wrangler": patch
+---
+
+fix: Update R2 Create Event Notification response
diff --git a/packages/wrangler/src/r2/helpers.ts b/packages/wrangler/src/r2/helpers.ts
@@ -523,8 +523,7 @@ export async function getEventNotificationConfig(
 
 /** Construct & transmit notification configuration to EWC.
  *
- * On success, receive HTTP 200 response with a body like:
- * { event_notification_detail_id: string }
+ * On success, receive HTTP 200 response with no body
  *
  * Possible status codes on failure:
  * - 400 Bad Request - Either:
@@ -541,7 +540,7 @@ export async function putEventNotificationConfig(
 	eventTypes: R2EventType[],
 	prefix?: string,
 	suffix?: string
-): Promise<{ event_notification_detail_id: string }> {
+): Promise<void> {
 	const queue = await getQueue(config, queueName);
 	const headers = eventNotificationHeaders(apiCredentials);
 	let actions: R2EventableOperation[] = [];
@@ -561,7 +560,7 @@ export async function putEventNotificationConfig(
 			" and "
 		)} (${actions.join(",")})`
 	);
-	return await fetchResult<{ event_notification_detail_id: string }>(
+	return await fetchResult<void>(
 		`/accounts/${accountId}/event_notifications/r2/${bucketName}/configuration/queues/${queue.queue_id}`,
 		{ method: "PUT", body: JSON.stringify(body), headers }
 	);