Fix default port and host overrides (#6810)

* Fix default port and host overrides
* Update package.json

Author: gabor-boros

net.coffee

@@ -94,12 +94,12 @@ aropt = util.aropt
 mkAtom = util.mkAtom
 mkErr = util.mkErr
 
-# These are the default hostname and port used by RethinkDB
-DEFAULT_HOST = 'localhost'
-DEFAULT_PORT = 28015
-
-module.exports.DEFAULT_HOST = DEFAULT_HOST
-module.exports.DEFAULT_PORT = DEFAULT_PORT
+# These are the default hostname and port used by RethinkDB.
+# Set the default hostname and port on the exported constants, to
+# keep backward compatibility when somebody would like to override
+# the default port or host name.
+module.exports.DEFAULT_HOST = 'localhost'
+module.exports.DEFAULT_PORT = 28015
 
 # These are strings returned by the server after a handshake
 # request. Since they must match exactly they're defined in
@@ -149,8 +149,8 @@ class Connection extends events.EventEmitter
             host = {host: host}
 
         # Here we set all of the connection parameters to their defaults.
-        @host = host.host || DEFAULT_HOST
-        @port = host.port || DEFAULT_PORT
+        @host = host.host || exports.DEFAULT_HOST
+        @port = host.port || exports.DEFAULT_PORT
 
         # One notable exception to defaulting is the db name. If the
         # user doesn't specify it, we leave it undefined. On the

package.json

@@ -1,6 +1,6 @@
 {
     "name": "rethinkdb",
-    "version": "2.4.0",
+    "version": "2.4.1",
     "main": "rethinkdb",
     "description": "This package provides the JavaScript driver library for the RethinkDB database server for use in your node application.",
     "keywords": ["database", "NoSQL", "reql", "query language"],