From 37b8ae7965a97a3105733db2b45f6472e3b9da6c Mon Sep 17 00:00:00 2001
From: Sean McArthur <sean.monstar@gmail.com>
Date: Thu, 19 Jul 2012 13:24:50 -0700
Subject: [PATCH] make statsd enabled default to false
defaulting on meant that even though it was
an optional dependency, it still blew chunks
---
lib/configuration.js | 2 +-
lib/statsd.js | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/lib/configuration.js b/lib/configuration.js
index 5d82f914d..d0bc838e0 100644
--- a/lib/configuration.js
+++ b/lib/configuration.js
@@ -127,7 +127,7 @@ var conf = module.exports = convict({
statsd: {
enabled: {
doc: "enable UDP based statsd reporting",
- format: 'boolean = true',
+ format: 'boolean = false',
env: 'ENABLE_STATSD'
},
host: "string?",
diff --git a/lib/statsd.js b/lib/statsd.js
index d18f24e75..20cc7dfdc 100644
--- a/lib/statsd.js
+++ b/lib/statsd.js
@@ -37,7 +37,7 @@ if (statsd_config && statsd_config.enabled) {
statsd = new StatsD(options["host"], options["port"]);
} else {
- log.error('statsd config enabled, but node-statsd not installed.');
+ logger.error('statsd config enabled, but node-statsd not installed.');
}
}
--
GitLab