Don't use `fs.readFile` in `sendFile()` method

[indexzero]

https://github.com/Pita/etherpad-lite/blob/master/node/server.js#L74

This is a bad idea because you're not sending any data to the request until the entire file is read. You could use the fs.createReadStream() and .pipe() APIs directly:

http://nodejs.org/docs/v0.4.7/api/fs.html#fs.createReadStream
http://nodejs.org/docs/v0.4.7/api/streams.html#stream.pipe

Or better yet, I would suggest using node-static instead. This is a battle-hardened static file server for node.js that we use everyday in production at Nodejitsu.

https://github.com/cloudhead/node-static

[Pita]

Yes It sounds like a good idea to use node-static. You wanna do that? I'm focusing on saving to the database at the moment

[indexzero]

Very busy leading up to JSConf / NodeConf next week. Checkout the docs for node-static, you should be able to handle it. Otherwise, I'll take a stab at this issue in mid-May

[Pita]

Do you have now time to do this? Would be great :)

[Pita]

I'm using now Express to serve the static files