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

Modernization and bugfixes #20

Open
wants to merge 12 commits into
from

Conversation

Projects
None yet
5 participants

tikonen commented Jan 29, 2013

Original library doesn't seem to be maintained anymore, but here are my changes anyway as we use this fork in production.

  • internal zlib (from fork by @cweider) for dynamic content
  • internal zlib for staticGzip
  • uses modern 'send' npm module to send static data
  • fixes race condition in case multiple node instances access same files
  • re-enabled HEAD request and it should now work properly as lib uses 'send' module

nateps commented on da6263a Sep 27, 2011

Good catch; stupid mistake. Just fixed it.

@owenb owenb referenced this pull request in socketstream/socketstream Jun 28, 2013

Merged

Added support for GZIP compression of static assets #376

@nateps, +1 to merging this. We're using @tikonen's fork as well... resolved our issues with the current NPM build.

@nateps +1 for merging this, also using @tikonen 's fork. Seems to be an issue with socket.io and gzip on Windows, getting "This socket is closed."

Automattic/socket.io#932

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