Various tweaks:

* Make options.port a number, after parsing the url
* Update some package.json dependencies (bar `ws`, left at 1.x)
* Make travis `script` execute `npm run ci` to send codecov results
* Add an .editorconfig file
* Add nodejs 4.3.2 to the build matrix for AWS Lambda
* Add C++11 compatible compiler for extensions

Author: sublimator

.editorconfig

@@ -0,0 +1,9 @@
+root = true
+
+[*]
+end_of_line = lf
+insert_final_newline = true
+charset = utf-8
+indent_style = space
+indent_size = 2
+trim_trailing_whitespace = true

.travis.yml

@@ -2,10 +2,23 @@ language: node_js
 sudo: false
 node_js:
 - '4'
+# AWS Lambda
+- '4.3.2'
 - '6'
 - '7'
+env:
+# For compiling optional extensions
+addons:
+  apt:
+    sources:
+      - ubuntu-toolchain-r-test
+    packages:
+      - g++-4.8
 env:
   global:
   - secure: ODwb1nuf12e0Ja/HgPwZh4aU01G8tTYZliSHI7ZWmYzGv3Yde6UnATeFG8sxGnWPRXTmaZelZfzhq7Aco1fEUnPm31af3z/iaV60iNmz6E1ifTR+oX7jxIvCDtHwBrgevrmIH8vrEUm/kQqDnFNrGG8Cc2xw/LjsNUG1wuWD+I4=
   - secure: buYrn01nPzsiduIQ5oqYTlBdDtM9WKP6gqoyq7IsutHb9sfwh9I6pUYsLibUo4Fq2um9QeXRZ4h1JLKK9xzDVSBpIGGaVzI4ClenfNt9O20IBGBnXcmEKPiRNYF4DkrqZzgx/OVWa6xzcRQI2R1ASQfoyfdpPAnqWXbfalSNkzs=
   - secure: IJRxzV03o76uiL4tCw/Zk0Es6tS/ATlQNIpQxZOyRLBoGTmZfZRKRxiESCKUASHudJgNIlw0kar2/LSJjMlYC4KnlrMJOLCYakXW+CWySe4q/f+qbrcdSK1+DZpjyr6Rmo654td/DD5KjNF3UgwBbi1GkE5fd4UL9HI5mPqDpqw=
+  - CXX=g++-4.8
+script:
+  - npm run ci

lib/connect/index.js

@@ -49,7 +49,13 @@ function connect (brokerUrl, opts) {
   opts = opts || {}
 
   if (brokerUrl) {
-    opts = xtend(url.parse(brokerUrl, true), opts)
+    var parsed = url.parse(brokerUrl, true)
+    if (parsed.port != null) {
+      parsed.port = Number(parsed.port)
+    }
+
+    opts = xtend(parsed, opts)
+
     if (opts.protocol === null) {
       throw new Error('Missing protocol')
     }

package.json

@@ -25,7 +25,7 @@
     "browser-build": "rimraf dist/ && mkdirp dist/ && browserify mqtt.js -s mqtt > dist/mqtt.js && uglifyjs --screw-ie8 < dist/mqtt.js > dist/mqtt.min.js",
     "browser-test": "zuul --server test/browser/server.js --local --open test/browser/test.js",
     "sauce-test": "zuul --server test/browser/server.js --tunnel ngrok -- test/browser/test.js",
-    "ci": "npm runt test && codecov"
+    "ci": "npm run test && codecov"
   },
   "pre-commit": [
     "test"
@@ -56,38 +56,38 @@
   },
   "dependencies": {
     "commist": "^1.0.0",
-    "concat-stream": "^1.4.7",
+    "concat-stream": "^1.6.0",
     "end-of-stream": "^1.1.0",
-    "help-me": "^1.0.0",
-    "inherits": "^2.0.1",
-    "minimist": "^1.1.0",
-    "mqtt-packet": "^5.0.0",
-    "readable-stream": "^2.1.0",
-    "pump": "^1.0.1",
-    "reinterval": "^1.0.1",
-    "split2": "^2.0.1",
-    "websocket-stream": "^3.3.2",
-    "xtend": "^4.0.0"
+    "help-me": "^1.0.1",
+    "inherits": "^2.0.3",
+    "minimist": "^1.2.0",
+    "mqtt-packet": "^5.2.1",
+    "readable-stream": "^2.2.3",
+    "pump": "^1.0.2",
+    "reinterval": "^1.1.0",
+    "split2": "^2.1.1",
+    "websocket-stream": "^3.3.3",
+    "xtend": "^4.0.1"
   },
   "devDependencies": {
-    "browserify": "^14.0.0",
+    "browserify": "^14.1.0",
     "codecov": "^1.0.1",
     "istanbul": "^0.4.5",
     "mkdirp": "^0.5.1",
     "mocha": "^3.2.0",
     "mqtt-connection": "^3.0.0",
     "nsp": "^2.6.2",
-    "pre-commit": "^1.1.3",
-    "rimraf": "^2.5.4",
+    "pre-commit": "^1.2.2",
+    "rimraf": "^2.6.1",
     "should": "*",
-    "sinon": "~1.17.0",
+    "sinon": "~1.17.7",
     "snazzy": "^6.0.0",
-    "standard": "^8.0.0",
-    "through2": "^2.0.0",
-    "uglify": "^0.1.1",
-    "uglify-js": "^2.7.4",
+    "standard": "^8.6.0",
+    "through2": "^2.0.3",
+    "uglify": "^0.1.5",
+    "uglify-js": "^2.7.5",
     "ws": "^1.0.0",
-    "zuul": "^3.4.0",
+    "zuul": "^3.11.1",
     "zuul-ngrok": "^4.0.0"
   },
   "standard": {

test/mqtt.js

@@ -57,7 +57,7 @@ describe('mqtt', function () {
       var c = mqtt.connect('tcp://user:pass@localhost:1883')
 
       c.options.should.have.property('hostname', 'localhost')
-      c.options.should.have.property('port', '1883')
+      c.options.should.have.property('port', 1883)
     })
 
     sslOpts = {