scottgonzalez / pretty-diff

colorized HTML diffs
MIT License
218 stars 27 forks source link

Issues with producing gists #7

Closed nhunzaker closed 11 years ago

nhunzaker commented 11 years ago

I love that this is an NPM package now!

Lately, we've been getting an issue whenever we try to create a gist-diff:

$ gist-diff master
GitHub username: nhunzaker
GitHub password for nhunzaker:

events.js:71
        throw arguments[1]; // Unhandled 'error' event
                       ^
Error: socket hang up
    at createHangUpError (http.js:1264:15)
    at CleartextStream.socketCloseListener (http.js:1315:23)
    at CleartextStream.EventEmitter.emit (events.js:126:20)
    at SecurePair.destroy (tls.js:938:22)
    at process.startup.processNextTick.process._tickCallback (node.js:244:9)

Any thoughts on this?

nhunzaker commented 11 years ago

Also, I have been able to produce this so far on node v0.8.14 and v0.10.12

scottgonzalez commented 11 years ago

I have some changes locally that I haven't finished which switch to using the new token authorization. I'll finish that up this week and look into this.

scottgonzalez commented 11 years ago

@nhunzaker Sorry this took so long to get to. I've been super busy, but I got some time to work on this while waiting in airports yesterday :-)

Let me know if this fixes the problem for you, then I'll do another release.

nhunzaker commented 11 years ago

@scottgonzalez This works like a charm. Thanks for taking the time to fix it!

scottgonzalez commented 11 years ago

Great. Just published 0.2.0 on npm.