wiki-cafe
/
node-oauth
0
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Merge pull request #211 from zedgu/master 

Fix #196 #207 (Removes some un-needed calls to tls/crypto)
This commit is contained in:
Ciaran Jessup 2014-08-08 14:31:35 +01:00
parent bfa43b44e7 47e50fe3ce
commit 69b0b7c3e8
2 changed files with 9 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
lib/oauth2.js
View File

@ -60,7 +60,6 @@ exports.OAuth2.prototype._chooseHttpLibrary= function( parsedUrl ) {
exports.OAuth2.prototype._request= function(method, url, headers, post_body, access_token, callback) {
var creds = crypto.createCredentials({ });
var parsedUrl= URL.parse( url, true );
if( parsedUrl.protocol == "https:" && !parsedUrl.port ) {
parsedUrl.port= 443;

10
tests/oauth2.js
View File

@ -1,6 +1,5 @@
var vows = require('vows'),
assert = require('assert'),
https = require('https'),
OAuth2= require('../lib/oauth2').OAuth2,
url = require('url');
@ -210,5 +209,14 @@ vows.describe('OAuth2').addBatch({
oa.get("", {});
}
}
},
'HTTPS URL connection testing, ': {
topic: function() {
var oa = new OAuth2("clientId", "clientSecret");
oa._request('GET', 'https://www.bing.com/', {}, null, '', this.callback);
},
'we should correctly get the response code == 200': function(error, result, response) {
assert.equal(response.statusCode, 200);
}
}
}).export(module);