Skip to content
This repository has been archived by the owner on Jun 4, 2023. It is now read-only.

Crash upon playing music #19

Closed
joermartin opened this issue May 3, 2014 · 5 comments
Closed

Crash upon playing music #19

joermartin opened this issue May 3, 2014 · 5 comments

Comments

@joermartin
Copy link

AirSonos will begin properly and listen for connections from airplay device. You can connect a device to AirSonos, such as iPhone but once audio is sent to the device, AirSonos crashes.

Below is the error log.

Sys-Admins-iMac:desktop sysAdmin$ airsonos
Searching for Sonos devices on network...
Setting up AirSonos for Kitchen {10.0.1.8:1400}

/usr/local/lib/node_modules/airsonos/node_modules/nodetunes/lib/rtsp.js:42
this.methodMapping[m.method](response, m.headers, m.content);
^
TypeError: Property '
SETUP' of object # is not a function
at RtspServer. (/usr/local/lib/node_modules/airsonos/node_modules/nodetunes/lib/rtsp.js:42:33)
at EventEmitter.emit (events.js:95:17)
at Socket. (/usr/local/lib/node_modules/airsonos/node_modules/nodetunes/node_modules/httplike/lib/parser.js:62:16)
at Socket.EventEmitter.emit (events.js:95:17)
at Socket. (stream_readable.js:745:14)
at Socket.EventEmitter.emit (events.js:92:17)
at emitReadable
(_stream_readable.js:407:10)
at emitReadable (_stream_readable.js:403:5)
at readableAddChunk (_stream_readable.js:165:9)
at Socket.Readable.push (_stream_readable.js:127:10)
Sys-Admins-iMac:desktop sysAdmin$

@stephen
Copy link
Owner

stephen commented May 3, 2014

Looks to be duplicate of issue #2. Can you reproduce this consistently? I'm not able to unfortunately..

@joermartin
Copy link
Author

I was able to reproduce consistently.. Until I went into the /usr/local and
changed nodeTunes rtsp.js file to include a trim method around the method
name. It now works. I am happy :-)

var methodTrimmed = m.method.trim();
this.methodMapping[methodTrimmed](response, m.headers, m.content);

I am not sure of why the server is sending the carriage return.

I am thinking that this is only happening for people who clean install,
/user/local is empty, and they install from git directly. That is my only
guess.

I hope that helps.
Joe

On Sat, May 3, 2014 at 6:59 PM, Stephen Wan notifications@github.comwrote:

Looks to be duplicate of issue #2#2.
Can you reproduce this consistently? I'm not able to unfortunately..


Reply to this email directly or view it on GitHubhttps://github.com//issues/19#issuecomment-42118716
.

@stephen
Copy link
Owner

stephen commented May 5, 2014

Can you walk me through exact repro steps? e.g. what devices you're streaming from, what sonos speakers you're streaming to, how you installed, etc. Why do you think /usr/local being empty changes things? Just want to glean more information about this problem.

I agree that the trim seems to do the trick in terms of fixing the issue, but I'm worried that there's an underlying problem with the way methods are being processed that might be hidden by the trim().

@stephen
Copy link
Owner

stephen commented May 10, 2014

Fixed in version 0.0.15.

@stephen stephen closed this as completed May 10, 2014
@joermartin
Copy link
Author

Just install it.. Works great. Keep up the great work

-Joe

On Fri, May 9, 2014 at 8:58 PM, Stephen Wan notifications@github.comwrote:

Closed #19 #19.


Reply to this email directly or view it on GitHubhttps://github.com//issues/19#event-119764084
.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants