fix(actions): pass actionAPIContext to action handler instead of APIC…

…ontext (#12301) Co-authored-by: Emanuele Stoppa <my.burning@gmail.com>
withastro · Oct 28, 2024 · 0cfc69d · 0cfc69d
1 parent ecc4402
commit 0cfc69d
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 2 deletions.
diff --git a/.changeset/shy-eggs-grin.md b/.changeset/shy-eggs-grin.md
@@ -0,0 +1,5 @@
+---
+'astro': patch
+---
+
+Fixes an issue with action handler context by passing the correct context (`ActionAPIContext`).
diff --git a/packages/astro/src/actions/runtime/middleware.ts b/packages/astro/src/actions/runtime/middleware.ts
@@ -99,7 +99,8 @@ async function handlePost({
 	if (contentType && hasContentType(contentType, formContentTypes)) {
 		formData = await request.clone().formData();
 	}
-	const action = baseAction.bind(context);
+	const { getActionResult, callAction, props, redirect, ...actionAPIContext } = context;
+	const action = baseAction.bind(actionAPIContext);
 	const actionResult = await action(formData);
 
 	if (context.url.searchParams.get(ACTION_QUERY_PARAMS.actionRedirect) === 'false') {

diff --git a/packages/astro/src/actions/runtime/route.ts b/packages/astro/src/actions/runtime/route.ts
@@ -27,7 +27,8 @@ export const POST: APIRoute = async (context) => {
 		// https://developer.mozilla.org/en-US/docs/Web/HTTP/Status/415
 		return new Response(null, { status: 415 });
 	}
-	const action = baseAction.bind(context);
+	const { getActionResult, callAction, props, redirect, ...actionAPIContext } = context;
+	const action = baseAction.bind(actionAPIContext);
 	const result = await action(args);
 	const serialized = serializeActionResult(result);