Skip to content

Fix for nodestunes crypto crash; 302. #42

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Kolky
Copy link

@Kolky Kolky commented Jun 19, 2016

This solves airsonos issue #302.
Using node v6.2.1 and npm 3.9.6.

@ntwb
Copy link

ntwb commented Jul 8, 2016

Thanks, this fixed AirSonos for me 👍

NPM v3.10.3, NodeJS v6.2.2 Mac OS X v10.11.5

@mrjcleaver
Copy link

Great, solved the problem for me too! Thank you :)

NPM 3.9.3 NodeJS 6.2.1 Mac OS X 10.10.5

@nickgs
Copy link

nickgs commented Jul 16, 2016

This worked great for me too. Thanks!

NPM 3.8.8 NodeJS v6.2.0 Mac OS X v10.11.5

@denisbr
Copy link

denisbr commented Jul 16, 2016

Works for me too, thanks.

@rib1
Copy link

rib1 commented Jul 18, 2016

works, thanks!

foomac:airsonos doe$ system_profiler SPSoftwareDataType
Software:

System Software Overview:

  System Version: OS X 10.11.5 (15F34)
  Kernel Version: Darwin 15.5.0
  Boot Volume: Macintosh HD
  Boot Mode: Normal
  Computer Name: foomac
  Secure Virtual Memory: Enabled
  System Integrity Protection: Enabled

foomac:airsonos doe$ node --version
v6.2.0
foomac:airsonos doe$ npm --version
3.10.3

@h9nry
Copy link

h9nry commented Jul 19, 2016

works, thanks!
node 6.3.0, npm 3.10.3, RaspberryPi 3

@richardtape
Copy link

Also allowed me to use airsonos. Couldn't be happier. Thanks so much!

npwalker added a commit to npwalker/airsonos that referenced this pull request Aug 3, 2016
@bengotow
Copy link

bengotow commented Aug 10, 2016

This is definitely the fix for the airsonos issue (worked for me at least!) Would be great to see this merged.

@alessiodionisi
Copy link

This fixed Nodetunes for me (macOS Sierra / iOS 10)

Please Pull!!

@zzamzoum
Copy link

Hello everyone
I am totally beginner and I try to install airsonos on my mac Sierra 10.12.
So I followed your advice .. but when I run youtube or VLC I get the following error: Error: Invalid key length
this is from the that I can not understand how you manage to settle the problem.
If someone has a little time to help me I would be grateful
thank you

@georgieapt
Copy link

Hi, Complete newbie to techie stuff also. Sorry, need a bit of a dummies guide here!
How/where am I inputting this fix?

Thanks, Georgie

@ntwb
Copy link

ntwb commented Nov 14, 2016

FYI: The new Sonos beta software version 7 now supports playing directly from Spotify 💥
So if you're here for the AIrSonos fix and you use Spotify you might no longer need this 🎉

@ernie932
Copy link

I also need help. I installed airsonos using this tutorial: https://hipsterpixel.co/2016/02/02/how-to-airplay-music-and-podcasts-to-your-sonos-speakers-tutorial/

I am not sure how to input this change. I have a Mac and am using Terminal. Do i use npm to install this fix? I am a total noob. Please help. I just need what command I need to put into Terminal to get this working.

@gismo141
Copy link

gismo141 commented Nov 21, 2016

Alright - How to install a working version:

First remove everything you've installed already:

npm uninstall -g airsonos
npm uninstall -g nodetunes

Then install the working forks:

npm install -g git://github.com/Kolky/nodetunes.git#master
cd ~
git clone https://github.com/lsmith77/airsonos.git airsonos
cd airsonos
npm install babel
npm install -g

Now everything should be installed so you now can execute that damn thing and hopefully it works:

airsonos

@ernie932
Copy link

Thank you! I will try this out when I get home.
One quick question: Why do I need to install babel? Is it something that airsonos needs to run properly?

@gismo141
Copy link

Yes somehow it is needed and written down as well in the dependencies.

I just reset my Mac so I was forced to reinstall everything and babel was one of the missing things.
Hopefully you're able to resolve the problem as well!

@ernie932
Copy link

Thank you so much! I was able to get it. Terminal gave me an error asking to run the command as root/Administrator. So I looked it up and ran the commands with sudo before it.

Thanks again!

@gismo141
Copy link

Glad it worked - have fun!

@burnflare
Copy link

This fixed airsonos for me! Any reason why this hasn't been merged into master yet?

@zzamzoum
Copy link

zzamzoum commented Dec 1, 2016 via email

@zzamzoum
Copy link

zzamzoum commented Dec 1, 2016 via email

@zzamzoum
Copy link

zzamzoum commented Dec 1, 2016 via email

@gismo141
Copy link

gismo141 commented Dec 1, 2016

It looks like you are executing the command in the wrong directory.

Please verify that you are executing it in the airsonos directory. You can verify the directory by issuing

pwd

@zzamzoum
Copy link

zzamzoum commented Dec 1, 2016 via email

@layzzzee8
Copy link

I get the same error

@JBalloonist
Copy link

Awesome this is working for me as well. Thank you!

@gismo141
Copy link

gismo141 commented Dec 5, 2016

@layzzzee8 and @zzamzoum normally you shouldn't need to use sudo to install these things.

If you need to, then maybe you could add --unsafe-perm to the npm install-calls.
E.g.

sudo npm install -g --unsafe-perm

See this Stack Overflow Topic.

Hopefully this helps.

@gpjt gpjt mentioned this pull request Jan 8, 2017
@russellbrenner
Copy link

Worked with the following procedure, as posted by @gismo141 above.

  • Remove existing install with:

npm uninstall -g airsono
npm uninstall -g nodetunes

  • Install from forks:

npm install -g git://github.com/Kolky/nodetunes.git#master
cd ~ (or whatever WD you want)
git clone https://github.com/lsmith77/airsonos.git airsonos
cd airsonos
npm install -g

Installs airsonos in /usr/local/bin/airsonos (global prefix assumed).

I've been running it with forever:

npm install -g forever
forever start -a -l forever.log -o airsonos.log -e airsonos_error.log /usr/local/bin/airsonos

System info:

System Version: macOS 10.12.2 (16C68)
Kernel Version: Darwin 16.3.0
Boot Mode: Normal
Secure Virtual Memory: Enabled
System Integrity Protection: Enabled

npm -v
4.0.5

node -v
v7.4.0

@sabrehagen
Copy link

@stephen Why is this not merged?

@HugoGresse
Copy link

@stephen Why is this not merged?

@jvanaert
Copy link

Hi @stephen would it be possible to merge this fix ? It would make the experience for new devs much easier to get things working with your nice libraries !
Or are you open for transferring the maintenance of this awesome lib into the community ?
Thanks !
Jeroen

mermaid pushed a commit to mermaid/nodetunes that referenced this pull request Mar 24, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.