🐛 id normalizes 127.0.0.1 to localhost

imlucas · imlucas · commit e6b8c4e6a932 · 2015-11-23T20:13:43.000-05:00
Fixes `Tried getting a connection to `localhost:27017` but it is not in deployment `localhost:27017`` https://bugsnag.com/mongodb/mongodb-scope-server/errors/56216df502c5eb75 2147026f
diff --git a/lib/model.js b/lib/model.js
@@ -139,5 +139,8 @@ module.exports.getId = function(id) {
   if (typeof id === 'number') {
     id = 'localhost:' + id;
   }
-  return id.toLowerCase().replace(hostname.toLowerCase(), 'localhost').replace('mongodb://', '');
+  return id.toLowerCase()
+    .replace(hostname.toLowerCase(), 'localhost')
+    .replace('127.0.0.1', 'localhost')
+    .replace('mongodb://', '');
 };
diff --git a/test/index.test.js b/test/index.test.js
@@ -23,5 +23,8 @@ describe('mongodb-instance-model', function() {
     it('should remove mongodb://', function() {
       assert.equal(Instance.getId('mongodb://localhost:27017'), 'localhost:27017');
     });
+    it('should substitute localhost for 127.0.0.1', function() {
+      assert.equal(Instance.getId('127.0.0.1:27017'), 'localhost:27017');
+    });
   });
 });
