Some WebSocket payload doesn't have "e" tag. about binance-rs-async HOT 4 CLOSED

This what WebsocketEventAlt and StreamEvent are for originally. You already have the receive the channel in 'StreamEvent' when you parse the event.
As you can see, there is another stream event you missed which is the partial depth stream that yields an OrderBook

from binance-rs-async.

I agree that having a single data type is superior, but this is at the cost of additional bytes that add no direct value since the information is already encoded in the enum type, let me try to hack a bit at your PR to see if we can get rid of this problem.

from binance-rs-async.

Yes, I didn't notice the StreamEvent and WebsocketEventAlt type. so I think the original implementation is fine.

from binance-rs-async.

Well, I think you are right design wise, but performance wise the original one will be slightly nimbler. However you made a good point so I will improve upon your PR and add a few methods in the websockets impl to add combined streams as well.

from binance-rs-async.