robrighter / node-boilerplate

Everything you need to get started on a Node.js website with express and socket.IO
1.05k stars 161 forks source link

Fix 404 error #7

Closed sgrove closed 12 years ago

sgrove commented 14 years ago

Seems like the ordering of the connect middleware and the server.router route for 404 ('/*') was preventing both routes and the staticProvider from working together (i.e. only one worked at a time). This patch should fix that so that when someone follows the README instructions, it just starts up.

slickplaid commented 14 years ago

This fix did not work for me. All static files are 404'd with and without this change for me. It's a fresh pull/install with no changes to any file.

I'm using node v0.3.1-pre and express v1.0.0rc4 on Ubuntu 10.04.1.

slickplaid commented 14 years ago

I re-cloned and manually cloned all the dependencies and it works great now. Not sure of what the issue was. This is without the pull request by the above author included.

Edit: Take that back. had to change socket.io's src path to /socket.io/socket.io.js instead of /js/socket.io.js. But, other than that it's working fine.

Thanks!

robrighter commented 12 years ago

This should be fixed now in version 2