Skip to content
New issue

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

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixed a bug: GET "/socket.io/socket.io.js.min" throws Error #423

Closed
wants to merge 1 commit into from
Closed

Fixed a bug: GET "/socket.io/socket.io.js.min" throws Error #423

wants to merge 1 commit into from

Conversation

sugyan
Copy link

@sugyan sugyan commented Jul 26, 2011

Manager.static.paths includes a key "/socket.io.js.min" and GET request for this PATH will serve "socket.io.min.js". but the PATH is "socket.io.js.min" that doesn't has mime type.
I fixed Manager not to serve static content for "socket.io.js.min".

@3rd-Eden
Copy link
Contributor

Thanks, but I have just rewritten the static backend for Socket.IO

See pull request: #391
But thanks for the effort and pull request anyways <3

@3rd-Eden 3rd-Eden closed this Jul 26, 2011
@sugyan
Copy link
Author

sugyan commented Jul 26, 2011

oh, I haven't seen the pull request pull/391 ...
ok, my pull request is unnecessary. thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants