alexwhitman / node-pushbullet-api

PushBullet API module for Node.js
165 stars 36 forks source link

Replace websocket with ws #41

Closed axfree closed 6 years ago

axfree commented 6 years ago

This seems to fix #40

alexwhitman commented 6 years ago

I've just tried this and it still seems to have the same issue, at least for me. I re-worked it a bit so that connect() started the connection and set up the event handlers. I'll do some more digging into what's going on.

alexwhitman commented 6 years ago

Actually, it does work, I just wasn't leaving it long enough. ws has a 30 second close timeout.

alexwhitman commented 6 years ago

Fixed in 2.3.0