CMR-10147: Removing query parameters from self url when generating Child links

src/__tests__/providerCatalog.spec.ts

@@ -164,6 +164,44 @@ describe("GET /:provider", () => {
         });
       });
     });
+
+    describe(`given the provider has a collection`, () => {
+      it("has a child link for that collection without query parameters", async function () {
+        this.timeout(5000);
+
+        sandbox
+          .stub(Provider, "getProviders")
+          .resolves([null, [{ "provider-id": "TEST", "short-name": "TEST" }]]);
+
+        const mockCollections = generateSTACCollections(1);
+        sandbox.stub(Collections, "getCollectionIds").resolves({
+          count: mockCollections.length,
+          cursor: "foundCursor",
+          items: mockCollections.map((coll) => ({
+            id: `${coll.shortName}_${coll.version}`,
+            title: coll.title ?? faker.random.words(4),
+          })),
+        });
+
+        const { body: catalog } = await request(stacApp).get("/stac/TEST?param=value");
+
+        const children = catalog.links.filter((l: Link) => l.rel === "child");
+        expect(children).to.have.length(mockCollections.length);
+
+        mockCollections.forEach((collection) => {
+          const childLink = children.find((l: Link) => l.href.endsWith(collection.id));
+
+          expect(childLink, JSON.stringify(children, null, 2)).to.have.property(
+            "type",
+            "application/json"
+          );
+          expect(childLink.href, JSON.stringify(childLink, null, 2)).to.not.contain("?param=value");
+          expect(childLink.href, JSON.stringify(childLink, null, 2)).to.match(
+            /^https?:\/\/.*TEST\/collections/
+          );
+        });
+      });
+    });
   });
 
   describe("given CMR providers endpoint responds with an error", () => {

src/__tests__/rootCatalog.spec.ts

@@ -55,7 +55,7 @@ describe("GET /stac", () => {
   });
 
   describe("given CMR responds with providers", () => {
-    before(() => {
+    beforeEach(() => {
       sandbox
         .stub(Providers, "getProviders")
         .resolves([null, [{ "provider-id": "TEST", "short-name": "TEST" }]]);
@@ -76,6 +76,23 @@ describe("GET /stac", () => {
         expect(providerLink.title).to.equal(provider["provider-id"]);
       });
     });
+
+    it("should have an entry for each provider in the links without query parameters", async () => {
+      const { statusCode, body } = await request(app).get("/stac?param=value");
+
+      expect(statusCode).to.equal(200);
+      const [, expectedProviders] = cmrProvidersResponse;
+
+      expectedProviders!.forEach((provider) => {
+        const providerLink = body.links.find((l: Link) => l.href.includes(provider["provider-id"]));
+
+        expect(providerLink.href).to.match(/^(http)s?:\/\/.*\w+/);
+        expect(providerLink.href).to.not.contain("?param=value");
+        expect(providerLink.rel).to.equal("child");
+        expect(providerLink.type).to.equal("application/json");
+        expect(providerLink.title).to.equal(provider["provider-id"]);
+      });
+    });
   });
 
   describe("given CMR providers endpoint responds with an error", () => {

src/routes/catalog.ts

@@ -105,7 +105,7 @@ export const providerCatalogHandler = async (req: Request, res: Response) => {
   const selfLinks = generateSelfLinks(req);
   const childLinks = (collections ?? []).map(({ id, title }) => ({
     rel: "child",
-    href: `${self}/collections/${encodeURIComponent(id)}`,
+    href: `${self.replace(/\?.*$/, "")}/collections/${encodeURIComponent(id)}`,
     title,
     type: "application/json",
   }));

src/routes/rootCatalog.ts

@@ -45,7 +45,7 @@ const providerLinks = (req: Request, providers: Provider[]): Link[] => {
     rel: "child",
     title,
     type: "application/json",
-    href: `${self}/${providerId}`,
+    href: `${self.replace(/\?.*$/, "")}/${providerId}`,
   }));
 };