Remove string filtering in logger

.circleci/config.yml

@@ -108,8 +108,7 @@ jobs:
     executor: docker-publisher
     steps:
       - checkout
-      - setup_remote_docker:
-          version: 20.10.6
+      - setup_remote_docker
       - run:
           name: Install Docker buildx
           command: |

package.json

@@ -367,5 +367,5 @@
     "qs": "6.9.7",
     "rollup": "^4.5.1"
   },
-  "version": "0.80.2"
+  "version": "0.80.2-ms-debug"
 }

plugins/azure/server/auth/azure.ts

@@ -8,6 +8,7 @@ import { slugifyDomain } from "@shared/utils/domains";
 import { parseEmail } from "@shared/utils/email";
 import accountProvisioner from "@server/commands/accountProvisioner";
 import { MicrosoftGraphError } from "@server/errors";
+import Logger from "@server/logging/Logger";
 import passportMiddleware from "@server/middlewares/passport";
 import { User } from "@server/models";
 import { AuthenticationResult } from "@server/types";
@@ -75,6 +76,18 @@ if (env.AZURE_CLIENT_ID && env.AZURE_CLIENT_SECRET) {
           );
         }
 
+        if (!organizationResponse.value) {
+          Logger.info(
+            "authentication",
+            "API response missing value key",
+            organizationResponse
+          );
+
+          throw MicrosoftGraphError(
+            "Unable to load organization value from Microsoft Graph API"
+          );
+        }
+
         const organization = organizationResponse.value[0];
 
         // Note: userPrincipalName is last here for backwards compatibility with

server/logging/Logger.ts

@@ -4,7 +4,6 @@ import chalk from "chalk";
 import isArray from "lodash/isArray";
 import isEmpty from "lodash/isEmpty";
 import isObject from "lodash/isObject";
-import isString from "lodash/isString";
 import winston from "winston";
 import env from "@server/env";
 import Metrics from "@server/logging/Metrics";
@@ -226,12 +225,6 @@ class Logger {
       return "[…]" as any as T;
     }
 
-    if (isString(input)) {
-      if (sensitiveFields.some((field) => input.includes(field))) {
-        return "[Filtered]" as any as T;
-      }
-    }
-
     if (isArray(input)) {
       return input.map((item) => this.sanitize(item, level + 1)) as any as T;
     }