Specify Content-Type as application/json unless specified in Accept

bcgov · Feb 12, 2025 · 8600e97 · 8600e97
1 parent 968d86d
commit 8600e97
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 1 deletion.
diff --git a/aws/tests/util.test.ts b/aws/tests/util.test.ts
@@ -23,12 +23,15 @@ describe("sanitizeHeaders", () => {
       applicationCd: "app123",
       correlationId: "12345",
       Accept: "application/octet-stream",
+      "Content-Type": "application/octet-stream",
     });
   });
 
   it("should return an empty object if no allowed headers are present", () => {
     const headers = { unauthorizedHeader: "shouldBeRemoved" };
-    expect(sanitizeHeaders(headers)).toEqual({});
+    expect(sanitizeHeaders(headers)).toEqual({
+      "Content-Type": "application/json",
+    });
   });
 });
 

diff --git a/aws/util.ts b/aws/util.ts
@@ -29,6 +29,12 @@ export const sanitizeHeaders = (
     }
   }
 
+  // Specify Content-Type as application/json unless specified in Accept
+  filteredHeaders["Content-Type"] =
+    filteredHeaders["Accept"] === "application/octet-stream"
+      ? "application/octet-stream"
+      : "application/json";
+
   return filteredHeaders;
 };