Revert Next.js server fetching URL checks that caused issues with Con…

…vex Auth (#33184) GitOrigin-RevId: 9e1738ce45f28b1acb71ecf04aa18144bcd7fcac
get-convex · Jan 15, 2025 · 6a1a7e7 · 6a1a7e7
1 parent 9ae2de6
commit 6a1a7e7
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 16 deletions.
diff --git a/npm-packages/convex/src/nextjs/index.ts b/npm-packages/convex/src/nextjs/index.ts
@@ -181,11 +181,6 @@ export async function fetchAction<Action extends FunctionReference<"action">>(
 }
 
 function setupClient(options: NextjsOptions) {
-  if ("url" in options && options.url === undefined) {
-    throw new Error(
-      "`url` property passed to convex/nextjs function but is undefined. Is an environment variable not set? If this is intended remove 'url' from the preloadQuery options.",
-    );
-  }
   const client = new ConvexHttpClient(
     getConvexUrl(options.url, options.skipConvexDeploymentUrlCheck ?? false),
   );

diff --git a/npm-packages/convex/src/nextjs/nextjs.test.tsx b/npm-packages/convex/src/nextjs/nextjs.test.tsx
@@ -19,17 +19,6 @@ describe("env setup", () => {
       "Environment variable NEXT_PUBLIC_CONVEX_URL is not set.",
     );
   });
-  test("requires url override to be present", async () => {
-    await expect(
-      preloadQuery(
-        anyApi.myQuery.default,
-        {},
-        { url: process.env.ENVAR_THAT_NEVER_EXISTS },
-      ),
-    ).rejects.toThrow(
-      "`url` property passed to convex/nextjs function but is undefined. Is an environment variable not set? If this is intended remove 'url' from the preloadQuery options.",
-    );
-  });
 });
 
 describe("preloadQuery and usePreloadedQuery", () => {