allow disabling of metrics endpoint

Signed-off-by: Travis Glenn Hansen <[email protected]>
travisghansen · Jun 16, 2023 · e7a49ed · e7a49ed
1 parent 10c5acc
commit e7a49ed
Show file tree

Hide file tree

Showing 3 changed files with 48 additions and 10 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,3 +1,9 @@
+# 0.13.1
+
+Released 2023-06-16
+
+- allow disabling of the metrices endpoint with env var `EAS_DISABLE_METRICS`
+
 # 0.13.0
 
 Released 2023-01-22

diff --git a/src/server.js b/src/server.js
@@ -44,17 +44,20 @@ const configTokenStoreManager = new ConfigTokenStoreManager(externalAuthServer);
 app.use(bodyParser.json());
 app.use(bodyParser.urlencoded({ extended: false }));
 app.use(cookieParser(externalAuthServer.secrets.cookie_sign_secret));
-app.use(
-  promBundle({
-    includeMethod: true,
-    includePath: true,
-    promClient: {
-      collectDefaultMetrics: {
-        timeout: 2000,
+
+if (!externalAuthServer.utils.toBoolean(process.env.EAS_DISABLE_METRICS)) {
+  app.use(
+    promBundle({
+      includeMethod: true,
+      includePath: true,
+      promClient: {
+        collectDefaultMetrics: {
+          timeout: 2000,
+        },
       },
-    },
-  })
-);
+    })
+  );
+}
 
 let revokedJtis = process.env["EAS_REVOKED_JTIS"];
 if (revokedJtis) {

diff --git a/src/utils.js b/src/utils.js
@@ -123,6 +123,34 @@ function generate_csrf_id() {
   return uuidv4();
 }
 
+function toBoolean(input) {
+  //return !!(dataStr?.toLowerCase?.() === 'true' || dataStr === true || Number.parseInt(dataStr, 10) === 0);
+  //return !!(dataStr?.toLowerCase?.() === 'true' || dataStr === true);
+  if (typeof input == "boolean") {
+    return input;
+  }
+
+  if (!isNaN(input)) {
+    if (typeof input == "string") {
+      input = parseFloat(input);
+    }
+    return Boolean(input);
+  }
+
+  if (typeof input == "string") {
+    switch (input.toLocaleLowerCase()) {
+      case "true":
+        return true;
+      case "false":
+        return false;
+      default:
+        return Boolean(input);
+    }
+  }
+
+  throw new Error("unable to determine boolean value");
+}
+
 function is_jwt(jwtString) {
   const re = new RegExp(
     /^[A-Za-z0-9-_=]+\.[A-Za-z0-9-_=]+\.?[A-Za-z0-9-_.+/=]*$/
@@ -506,6 +534,7 @@ module.exports = {
   base64_decode,
   generate_session_id,
   generate_csrf_id,
+  toBoolean,
   is_json_like,
   is_yaml_like,
   is_jwt,