Merge pull request #366 from zdoc01/strictssl-option

Add support for strictSSL request option

README.md

@@ -21,6 +21,7 @@ var T = new Twit({
   access_token:         '...',
   access_token_secret:  '...',
   timeout_ms:           60*1000,  // optional HTTP request timeout to apply to all requests.
+  strictSSL:            true,     // optional - requires SSL certificates to be valid.
 })
 
 //

lib/twitter.js

@@ -206,6 +206,10 @@ Twitter.prototype._buildReqOpts = function (method, path, params, isStreaming, c
     reqOpts.timeout = self.config.timeout_ms;
   }
 
+  if (typeof self.config.strictSSL !== 'undefined') {
+    reqOpts.strictSSL = self.config.strictSSL;
+  }
+
   try {
     // finalize the `path` value by building it using user-supplied params
     path = helpers.moveParamsIntoPath(finalParams, path)
@@ -479,6 +483,10 @@ Twitter.prototype._validateConfigOrThrow = function (config) {
     throw new TypeError('Twit config `timeout_ms` must be a Number. Got: ' + config.timeout_ms + '.');
   }
 
+  if (typeof config.strictSSL !== 'undefined' && typeof config.strictSSL !== 'boolean') {
+    throw new TypeError('Twit config `strictSSL` must be a Boolean. Got: ' + config.strictSSL + '.');
+  }
+
   if (config.app_only_auth) {
     var auth_type = 'app-only auth'
     var required_keys = required_for_app_auth

tests/twit.js

@@ -63,6 +63,20 @@ describe('twit', function () {
 
       done()
     })
+
+    it('throws when config provides invalid strictSSL option', function (done) {
+      assert.throws(function () {
+        var twit = new Twit({
+            consumer_key: 'a'
+          , consumer_secret: 'a'
+          , access_token: 'a'
+          , access_token_secret: 'a'
+          , strictSSL: 'a'
+        })
+      }, Error)
+
+      done()
+    })
   })
 
   describe('setAuth()', function () {