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

Fix #465 #470

Merged
merged 1 commit into from
May 3, 2017
Merged

Fix #465 #470

merged 1 commit into from
May 3, 2017

Conversation

zhoulifu
Copy link
Contributor

@zhoulifu zhoulifu commented May 3, 2017

Prevent decoding handshake response after closing the WebSocketImpl

Prevent decoding handshake response after client closed
@zhoulifu
Copy link
Contributor Author

zhoulifu commented May 3, 2017

@Irahe PTAL

@Irahe
Copy link

Irahe commented May 3, 2017

Hello @zhoulifu , i'll merge this code to my lib and rebuild to test...
Thank you for your time.
i'm realy proud of this community!

👍

@Irahe
Copy link

Irahe commented May 3, 2017

Dear @zhoulifu , Unfortunately the fix wont work for me. I still have the same issues on firefox and Edge.

@zhoulifu
Copy link
Contributor Author

zhoulifu commented May 3, 2017

OMG, It's a mistaken identity, i mean to @marci4, HAHA
Sorry for disturbing you @Irahe XD

@marci4 marci4 merged commit 56e0d75 into TooTallNate:master May 3, 2017
@marci4 marci4 added this to the Release 1.3.4 milestone May 3, 2017
@zhoulifu zhoulifu deleted the issue465 branch May 4, 2017 15:50
@zhoulifu zhoulifu restored the issue465 branch May 4, 2017 15:50
@zhoulifu zhoulifu deleted the issue465 branch May 4, 2017 15:51
@marci4 marci4 added the Bug label May 23, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants