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

Fixed bug #128 #129

Closed
wants to merge 2 commits into
from

Conversation

Projects
None yet
2 participants
Contributor

leg0 commented Oct 22, 2012

No description provided.

Do you have a test case we can add to the test suite to prevent regressions on this bug?

Owner

leg0 replied Oct 22, 2012

No, but I could make one. Where should that test be? Is file t2/app_test.erl and function test_websocket/0 a good place?

Yes, t2/app_test.erl, function test_websocket/0.

Meanwhile, I ran the autobahn fuzzingclient tests with your changes and it doesn't break anything. That's good. :)

Contributor

leg0 commented Oct 23, 2012

I added the test case.

Collaborator

vinoski commented Oct 23, 2012

I manually pulled your changes to my clone, squashed your two commits, and pushed the changes to github master. Thanks for the fixes!

@vinoski vinoski closed this Oct 23, 2012

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