Open steforster opened 10 years ago
Ah this fixes that weird https://github.com/Olivine-Labs/Alchemy-Websockets/blob/master/src/Alchemy/WebSocketClient.cs#L151 stuff; I'm not sure why that actually would ever work. Thanks, please merge.
I've tested this fork it's running supper great, no missing packet anymore. Please merge!
I picked Alchemy as communication framework for my remote actor library because of Alchemys slim design and the availability of a client implementation.
But after some initial success I ran into more and more bugs, especially on the client side. This pull request collects all committs that were needed to make the library much more stable. When reading the list of open issues I think some of them could be fixed by this pull request too: E.g. #93, #89, #84, #78, #72, #66, #60, #53, #49, #25
The changed points are:
Fixed errors and made it rfc6455 conformant
Fixed message loss, when several messages are received in the same TCP packet
Useful when a server must be very responsive. When setting this flag, the maximal throughput rises from 33 to more than 5000 request/response pairs per second