fix(next): Avoid appending ? if querystring is empty (#71)

Co-authored-by: David Mytton <[email protected]>

arcjet-next/index.ts

@@ -145,9 +145,12 @@ export default function arcjetNext<const Rules extends (Primitive | Product)[]>(
       const method = request.method ?? "";
       const host = headers.get("host") ?? "";
       let path;
-      // TODO(#224): nextUrl has formatting logic when you `toString` but we don't account for that here
+      // TODO(#36): nextUrl has formatting logic when you `toString` but we don't account for that here
       if (typeof request.nextUrl !== "undefined") {
-        path = request.nextUrl.pathname + "?" + request.nextUrl.search;
+        path = request.nextUrl.pathname;
+        if (request.nextUrl.search !== "") {
+          path += "?" + request.nextUrl.search;
+        }
       } else {
         path = request.url ?? "";
       }

examples/nextjs-14-app-dir-rl/middleware.ts

@@ -0,0 +1,10 @@
+import { createMiddleware, rateLimit } from "@arcjet/next";
+export const config = {
+  // matcher tells Next.js which routes to run the middleware on
+  matcher: ["/"],
+};
+const middleware = createMiddleware({
+  key: "ajkey_yourkey",
+  rules: [],
+});
+export default middleware;