fix: fix basePath support for API routes (#523)

* fix: fix basePath support for API routes Fixes: #522 * Create polite-news-happen.md --------- Co-authored-by: conico974 <nicodorseuil@yahoo.fr>
opennextjs · Oct 3, 2024 · 9fceedb · 9fceedb
1 parent 50703a3
commit 9fceedb
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 2 deletions.
diff --git a/.changeset/polite-news-happen.md b/.changeset/polite-news-happen.md
@@ -0,0 +1,5 @@
+---
+"@opennextjs/aws": patch
+---
+
+fix: fix basePath support for API routes
diff --git a/packages/open-next/src/core/routingHandler.ts b/packages/open-next/src/core/routingHandler.ts
@@ -34,6 +34,11 @@ const optionalBasepathPrefixRegex = !!RoutesManifest.basePath
   ? `^${RoutesManifest.basePath}/?`
   : "^/";
 
+// Add the basePath prefix to the api routes
+const apiPrefix = !!RoutesManifest.basePath
+  ? `${RoutesManifest.basePath}/api`
+  : "/api";
+
 const staticRegexp = RoutesManifest.routes.static.map(
   (route) =>
     new RegExp(
@@ -145,8 +150,8 @@ export default async function routingHandler(
   // /api even if it's a page route doesn't get generated in the manifest
   // Ideally we would need to properly check api routes here
   const isApiRoute =
-    internalEvent.rawPath === "/api" ||
-    internalEvent.rawPath.startsWith("/api/");
+    internalEvent.rawPath === apiPrefix ||
+    internalEvent.rawPath.startsWith(`${apiPrefix}/`);
 
   const isNextImageRoute = internalEvent.rawPath.startsWith("/_next/image");