passenger-nodejs-websocket-demo
passenger-nodejs-websocket-demo copied to clipboard
Add Vagrant
This adds a Vagrant box which runs the demo on Nginx. If this sounds like a good idea I can go ahead and configure additional boxes for Apache and Standalone.
Yes this is a good idea. A few comments though:
- I think Vagrant-specific resource files, like the Nginx config file, should be placed in a directory named
devorvagrantin order to signal that they're only supposed to be used during development, not production. - The provisioning shell script should be stored in an external file so that syntax highlighting works properly.
- The provisioning shell script should be idempotent. It looks like a reprovision would fail on the
lncalls.
Cool, I'll take a look at those things when I get home today.