Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Crash on spotify login #79

Closed
dimfeld opened this issue Feb 7, 2014 · 7 comments · Fixed by #91
Closed

Crash on spotify login #79

dimfeld opened this issue Feb 7, 2014 · 7 comments · Fixed by #91

Comments

@dimfeld
Copy link

dimfeld commented Feb 7, 2014

I'm seeing the following crash after I click "Submit" on the Spotify login page. I'm using the latest master branch code.

When running npm install, I got this version of spotify-web: spotify-web@0.1.2 node_modules/spotify-web

Any ideas? Let me know if there's some debug output I can enable that will help you to debug this futher.

portify 0.51
Starting without harmony
info - socket.io started
Google Login success
ping_flash2 [ '249 169 26 68 67 31 243 106 20 93 160 137 198 186 41 91 20 197 220 80' ]

Error: TODO: implement!
at Spotify._onmessagecommand (/Users/dimfeld/src/portify/data/node_modules/spotify-web/lib/spotify.js:389:11)
at Spotify.EventEmitter.emit (events.js:98:17)
at Spotify._onmessage (/Users/dimfeld/src/portify/data/node_modules/spotify-web/lib/spotify.js:358:10)
at WebSocket.EventEmitter.emit (events.js:98:17)
at Receiver.self._receiver.ontext (/Users/dimfeld/src/portify/data/node_modules/spotify-web/node_modules/ws/lib/WebSocket.js:697:10)
at Receiver.opcodes.1.finish (/Users/dimfeld/src/portify/data/node_modules/spotify-web/node_modules/ws/lib/Receiver.js:397:14)
at Receiver.expectHandler (/Users/dimfeld/src/portify/data/node_modules/spotify-web/node_modules/ws/lib/Receiver.js:384:31)
at Receiver.add (/Users/dimfeld/src/portify/data/node_modules/spotify-web/node_modules/ws/lib/Receiver.js:93:24)
at CleartextStream.firstHandler (/Users/dimfeld/src/portify/data/node_modules/spotify-web/node_modules/ws/lib/WebSocket.js:678:22)
at CleartextStream.EventEmitter.emit (events.js:95:17)

@dimfeld
Copy link
Author

dimfeld commented Feb 7, 2014

I worked around this by editing line 384 of portify/data/nodemodules/spotify-web/lib/spotify.js to ignore the ping_flash2 event. This got me on to the transfer page, which seems to be working so far.

-} else if ('login_complete' == command) {
+} else if ('login_complete' == command || 'ping_flash2' == command) {

@ptrics
Copy link

ptrics commented Feb 8, 2014

can you please repack this for use in windows, 'cause i have the exact same error and can not pass to the step 3 (to watch the playlists)

@ptrics
Copy link

ptrics commented Feb 8, 2014

nevermind i figure it out with free java script editor.
thanks

@vandepoelm
Copy link

Hi, i had this problem too, fixed it with the code provided by dimfeld.

@the-lay
Copy link

the-lay commented Feb 15, 2014

Confirming, just had that bug too. That's spotify-web lib bug, it already has open issue.
And also confirming that @dimfeld 's workaround helped.

@mauimauer
Copy link
Owner

And since that issue was closed recently....

@ErikBjare
Copy link
Contributor

@mauimauer Well it was closed but your "spotify-web" ref in package.json points to mauimauer/node-spotify-web which is still v0.1.2 and does not have the fix. Simply changing the custom tarball path in package.json to * worked for me, I've had no backwards-compability issues.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
6 participants