Merge pull request #80 from groupon/jk-less-aggressive-search-domain

Ignore searchDomain for localhost and IP

lib/fetch.js

@@ -177,6 +177,24 @@ function defaultStatusCode(value, defaultValue) {
   return defaultValue;
 }
 
+var IPv4 = /^\d{1,3}.\d{1,3}.\d{1,3}.\d{1,3}$/;
+function canApplySearchDomain(hostname) {
+  if (hostname === 'localhost') return false;
+  if (hostname[hostname.length - 1] === '.') return false;
+  // A hostname shouldn't contain ":" (IPv6 adddress) or be an IPv4 address
+  return hostname.indexOf(':') === -1 && !IPv4.test(hostname);
+}
+
+function buildHostname(hostname, searchDomain) {
+  if (!hostname || typeof hostname !== 'string') {
+    throw new Error('Invalid URI ' + JSON.stringify(hostname));
+  }
+  if (searchDomain && canApplySearchDomain(hostname)) {
+    return hostname + '.' + searchDomain + '.';
+  }
+  return hostname;
+}
+
 function fetchUrlObj(urlObj, options) {
   if (options.baseUrl && typeof options.baseUrl === 'string') {
     urlObj = applyBaseUrl(urlObj, options.baseUrl);
@@ -204,14 +222,7 @@ function fetchUrlObj(urlObj, options) {
     body = qsParser.stringify(form);
   }
 
-  var hostname = urlObj.hostname;
-  if (!hostname || typeof hostname !== 'string') {
-    throw new Error('Invalid URI ' + JSON.stringify(hostname));
-  }
-
-  if (options.searchDomain && hostname[hostname.length - 1] !== '.') {
-    hostname += '.' + options.searchDomain + '.';
-  }
+  var hostname = buildHostname(urlObj.hostname, options.searchDomain);
 
   var agent = getAgent(options, urlObj);
   assign(agent.options, {

test/fetch-search-domain.test.js

@@ -1,5 +1,7 @@
 'use strict';
 
+var http = require('http');
+
 var assert = require('assertive');
 
 var fetch = require('../').fetch;
@@ -64,4 +66,38 @@ describe('fetch: searchDomain', function() {
       }
     });
   });
+
+  describe('localhost and IP', function() {
+    var server = http.createServer(function(req, res) {
+      res.end('{"ok":true}');
+    });
+
+    before(function(done) {
+      server.listen(done);
+    });
+
+    it('never appends the searchDomain to localhost', function() {
+      var options = {
+        baseUrl: 'http://localhost:' + server.address().port,
+        searchDomain: 'bar123',
+      };
+      return fetch('/path', options)
+        .json()
+        .then(function(result) {
+          assert.deepEqual({ ok: true }, result);
+        });
+    });
+
+    it('never appends the searchDomain to an IP address', function() {
+      var options = {
+        baseUrl: 'http://127.0.0.1:' + server.address().port,
+        searchDomain: 'bar123',
+      };
+      return fetch('/path', options)
+        .json()
+        .then(function(result) {
+          assert.deepEqual({ ok: true }, result);
+        });
+    });
+  });
 });