Make the /metrics page disableable with $DISABLE_METRICS

mozilla-services · Nov 11, 2016 · a18437a · a18437a
1 parent fd4a512
commit a18437a
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 1 deletion.
diff --git a/server/src/config.js b/server/src/config.js
@@ -174,6 +174,13 @@ var conf = convict({
     env: "REFRESH_METRICS_TIME",
     arg: "refresh-metrics-time"
   },
+  disableMetrics: {
+    doc: "If true, do not mount /metrics or start jobs",
+    format: Boolean,
+    default: false,
+    env: "DISABLE_METRICS",
+    arg: "disable-metrics"
+  },
   allowExport: {
     doc: "Whether to allow exporting shots",
     format: Boolean,

diff --git a/server/src/server.js b/server/src/server.js
@@ -969,7 +969,9 @@ app.use(function (err, req, res, next) {
   errorResponse(res, "General error:", err);
 });
 
-app.use("/metrics", require("./pages/metrics/server").app);
+if (! config.disableMetrics) {
+  app.use("/metrics", require("./pages/metrics/server").app);
+}
 
 app.use("/shots", require("./pages/shotindex/server").app);