[D1] replace D1's dashed time-travel endpoints with underscored ones (#…

…4841) * [D1] replace D1's dashed time-travel endpoints with underscored ones * [D1] make linter happy
cloudflare · Jan 26, 2024 · 1039612 · 1039612
1 parent b3c5566
commit 1039612
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 2 deletions.
diff --git a/.changeset/fifty-pigs-behave.md b/.changeset/fifty-pigs-behave.md
@@ -0,0 +1,7 @@
+---
+"wrangler": patch
+---
+
+fix: replace D1's dashed time-travel endpoints with underscored ones
+
+D1 will maintain its `d1/database/${databaseId}/time-travel/*` endpoints until GA, at which point older versions of wrangler will start throwing errors to users, asking them to upgrade their wrangler version to continue using Time Travel via CLI.
diff --git a/packages/wrangler/src/d1/timeTravel/restore.ts b/packages/wrangler/src/d1/timeTravel/restore.ts
@@ -101,7 +101,7 @@ const handleRestore = async (
 	searchParams: URLSearchParams
 ) => {
 	return await fetchResult<RestoreBookmarkResponse>(
-		`/accounts/${accountId}/d1/database/${databaseId}/time-travel/restore?${searchParams.toString()}`,
+		`/accounts/${accountId}/d1/database/${databaseId}/time_travel/restore?${searchParams.toString()}`,
 		{
 			headers: {
 				"Content-Type": "application/json",

diff --git a/packages/wrangler/src/d1/timeTravel/utils.ts b/packages/wrangler/src/d1/timeTravel/utils.ts
@@ -22,7 +22,7 @@ export const getBookmarkIdFromTimestamp = async (
 	}
 
 	const bookmarkResult = await fetchResult<BookmarkResponse>(
-		`/accounts/${accountId}/d1/database/${databaseId}/time-travel/bookmark?${searchParams.toString()}`,
+		`/accounts/${accountId}/d1/database/${databaseId}/time_travel/bookmark?${searchParams.toString()}`,
 		{
 			headers: {
 				"Content-Type": "application/json",