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

Already on GitHub? Sign in to your account

library doesn't work on Heroku #29

Closed
jpgarcia opened this Issue Jan 11, 2012 · 4 comments

Comments

Projects
None yet
2 participants

I tried to run the library on Heroku without success since it only supports node v0.4.7 and the azure npm package engine is set to >= 0.6.4

I forked this repo and published an npm package 'azure-0.4.7' to perform a test on Heroku and it seems to be working fine.

is there any reason to keep the v0.6.4 as the minimun required version? can you change it to 0.4.7?

thanks
.jpg

Hi @jpgarcia

We discussed this. The main reason why we have v0.6.4 was that there were a few problematic node.exe releases that would cause issues with the libraries. However, we do see the value of beeing retrocompatible with some older versions where the libraries do work. Therefore, we'll change it.

Thank you for raising this issue

@ghost ghost assigned andrerod Jan 11, 2012

Great! thank you for your quick answer

Meanwhile, I wrote a blog post here http://nodeblog.cloudapp.net/using-the-windows-azure-sdk-for-nodejs-on-heroku

thanks!

@jpgarcia thanks for the post. It's however worth noticing that although we are making the requirement less strict, the correct solution for this would probably be for heroku to move to a more recent node version. Since version 0.4.7 there have been multiple changes that could affect not only this, but other libraries. As I mentioned, there are even version > 0.4.7 and < 0.6.4 where the libraries wont even work correctly as they should due to node / node.exe issues.

Also, and although I haven't tried it myself, it seems to be possible to run your own version of node in heroku:
http://blog.superpat.com/2011/11/15/running-your-own-node-js-version-on-heroku/

Probably that would be a much better and reliable solution than relying on an outdated engine version.

Should now be fixed on dev branch.

@andrerod andrerod closed this Jan 16, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment