Skip to content
This repository has been archived by the owner on Nov 28, 2017. It is now read-only.

Commit

Permalink
Replaced HTTPS with HTTP to fix connection error
Browse files Browse the repository at this point in the history 
This module does not handle user credentials.

> Error: 140735110496640:error:0607907F:digital envelope routines:EVP_PKEY_get1_RSA:expecting an rsa key:../deps/openssl/openssl/crypto/evp/p_lib.c:288:
  • Loading branch information
@fvdm
fvdm committed Mar 1, 2013
1 parent 1caea4f commit 1e34663
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions youtube-feeds.js
View file
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@ OTHER DEALINGS IN THE SOFTWARE.
For more information, please refer to <http://unlicense.org>
*/

var https = require('https'),
var http = require('http'),
xml2json = require('node-xml2json'),
querystring = require('querystring')

Expand Down Expand Up @@ -218,7 +218,7 @@ app.talk = function( path, fields, cb, oldJsonKey ) {
// prepare
var options = {
hostname: 'gdata.youtube.com',
port: 443,
port: 80,
path: '/'+ path +'?'+ querystring.stringify( fields ),
headers: {
'User-Agent': 'youtube-feeds.js (https://github.com/fvdm/nodejs-youtube)',
Expand All @@ -228,7 +228,7 @@ app.talk = function( path, fields, cb, oldJsonKey ) {
}

// request
var request = https.request( options, function( response ) {
var request = http.request( options, function( response ) {

// response
var data = ''
Expand Down

1 comment on commit 1e34663

@fvdm
Copy link
Owner Author

@fvdm fvdm commented on 1e34663 Mar 16, 2013

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This 'fix' is more a workaround, due to lack of time I have not been able to correctly trace the bug. At lease people can continue to use the module.

According to the discussion in nodejs/node-v0.x-archive#4771 it appears to be a Node bug.

Please sign in to comment.