Authoriser

Author: stevebux

.vscode/settings.json

@@ -10,6 +10,5 @@
     "**/Thumbs.db": true,
     ".github": false,
     ".vscode": false
-  },
-  "typescript.tsdk": "node_modules/typescript/lib"
+  }
 }

internal/helpers/src/__tests__/logger.test.ts

@@ -12,13 +12,4 @@ describe("createLogger", () => {
 
     expect(logger.level).toBe("debug");
   });
-
-  it("should have expected logging methods", () => {
-    const logger = createLogger();
-
-    expect(typeof logger.info).toBe("function");
-    expect(typeof logger.error).toBe("function");
-    expect(typeof logger.warn).toBe("function");
-    expect(typeof logger.debug).toBe("function");
-  });
 });

lambdas/api-handler/src/config/__tests__/deps.test.ts

@@ -15,7 +15,6 @@ describe("createDependenciesContainer", () => {
     jest.clearAllMocks();
     jest.resetModules();
 
-    // @internal/helpers - createLogger
     jest.mock("@internal/helpers", () => ({
       createLogger: jest.fn(() => ({
         info: jest.fn(),

lambdas/authorizer/src/authorizer.ts

@@ -18,8 +18,6 @@ export default function createAuthorizerHandler(
     context: Context,
     callback: Callback<APIGatewayAuthorizerResult>,
   ): void => {
-    deps.logger.info({ description: "Received authorizer event", event });
-
     checkCertificateExpiry(event.requestContext.identity.clientCert, deps);
 
     getSupplier(event.headers, deps)
@@ -135,9 +133,9 @@ async function checkCertificateExpiry(
 ): Promise<void> {
   deps.logger.info({
     description: "Client certificate details",
-    issuerDN: certificate?.issuerDN,
-    subjectDN: certificate?.subjectDN,
-    validity: certificate?.validity,
+    issuerDN: certificate?.issuerDN || "-",
+    subjectDN: certificate?.subjectDN || "-",
+    validity: certificate?.validity || "-",
   });
 
   if (!certificate) {