fix: do not add query params on top of the URL with query params

cloudflare · Nov 20, 2024 · ea37472 · ea37472
1 parent 3e532ba
commit ea37472
Showing 1 changed file with 7 additions and 2 deletions.
diff --git a/src/router.ts b/src/router.ts
@@ -84,11 +84,13 @@ v2Router.delete("/:name+/manifests/:reference", async (req, env: Env) => {
     }
   }
 
+  const url = new URL(req.url);
   if (tags.truncated) {
+    url.searchParams.set("last", tags.truncated ? tags.cursor : "");
     return new Response(JSON.stringify(ManifestTagsListTooBigError), {
       status: 400,
       headers: {
-        "Link": `${req.url}/?last=${tags.truncated ? tags.cursor : ""}; rel=next`,
+        "Link": `${url.toString()}; rel=next`,
         "Content-Type": "application/json",
       },
     });
@@ -529,6 +531,9 @@ v2Router.get("/:name+/tags/list", async (req, env: Env) => {
   });
 
   const keys = tags.objects.map((object) => object.key.split("/").pop()!);
+  const url = new URL(req.url);
+  url.searchParams.set("n", `${n}`);
+  url.searchParams.set("last", keys.length ? keys[keys.length - 1] : "");
   return new Response(
     JSON.stringify({
       name,
@@ -538,7 +543,7 @@ v2Router.get("/:name+/tags/list", async (req, env: Env) => {
       status: 200,
       headers: {
         "Content-Type": "application/json",
-        "Link": `${req.url}?n=${n}&last=${keys.length ? keys[keys.length - 1] : ""}; rel=next`,
+        "Link": `${url.toString()}; rel=next`,
       },
     },
   );