Case insensitive enablement env var

Author: CarlesDD

src/appsec/index.spec.ts

@@ -62,6 +62,38 @@ describe("AppSec orchestrator", () => {
       expect(mockPublish).toHaveBeenCalled();
     });
 
+    it("should enable when DD_APPSEC_ENABLED is TRUE (uppercase)", () => {
+      process.env.DD_APPSEC_ENABLED = "TRUE";
+      initAppsec();
+
+      const span = { setTag: jest.fn() };
+      mockExtract.mockReturnValue({
+        headers: {},
+        method: "GET",
+        path: "/",
+        isBase64Encoded: false,
+      });
+
+      processAppsecRequest({}, span);
+      expect(mockPublish).toHaveBeenCalled();
+    });
+
+    it("should enable when DD_APPSEC_ENABLED is True (mixed case)", () => {
+      process.env.DD_APPSEC_ENABLED = "True";
+      initAppsec();
+
+      const span = { setTag: jest.fn() };
+      mockExtract.mockReturnValue({
+        headers: {},
+        method: "GET",
+        path: "/",
+        isBase64Encoded: false,
+      });
+
+      processAppsecRequest({}, span);
+      expect(mockPublish).toHaveBeenCalled();
+    });
+
     it("should not enable when DD_APPSEC_ENABLED is not set", () => {
       delete process.env.DD_APPSEC_ENABLED;
       initAppsec();
@@ -77,6 +109,14 @@ describe("AppSec orchestrator", () => {
       processAppsecRequest({}, {});
       expect(mockPublish).not.toHaveBeenCalled();
     });
+
+    it("should not enable when DD_APPSEC_ENABLED is FALSE (uppercase)", () => {
+      process.env.DD_APPSEC_ENABLED = "FALSE";
+      initAppsec();
+
+      processAppsecRequest({}, {});
+      expect(mockPublish).not.toHaveBeenCalled();
+    });
   });
 
   describe("processAppSecRequest", () => {

src/appsec/index.ts

@@ -9,7 +9,7 @@ const endInvocationChannel = dc.channel("datadog:lambda:end-invocation");
 let enabled = false;
 
 export function initAppsec(): void {
-  const envValue = process.env.DD_APPSEC_ENABLED;
+  const envValue = process.env.DD_APPSEC_ENABLED?.toLowerCase();
   enabled = envValue === "true" || envValue === "1";
 }