HOSTNAME and HOST precedence

The HOSTNAME and HOST ENV variables now take precedence over OS.hostname()
node-config · Nov 6, 2012 · 5060b04 · 5060b04
1 parent dd726e5
commit 5060b04
Showing 1 changed file with 7 additions and 3 deletions.
diff --git a/lib/config.js b/lib/config.js
@@ -578,10 +578,14 @@ Config.prototype._loadFileConfigs = function() {
   // Determine the host name from the OS module, $HOST, or $HOSTNAME
   // Remove any . appendages, and default to null if not set
   try {
-    var OS = require('os');
-    var hostName = OS.hostname();
+    var hostName = process.env.HOST || process.env.HOSTNAME;
+
+    if (!hostName) {
+        var OS = require('os');
+        hostName = OS.hostname();
+    }
   } catch (e) {
-    hostName = process.env.HOST || process.env.HOSTNAME;
+    hostName = '';
   }
   hostName = hostName ? hostName.split('.')[0] : null;