Async server example and constructor with rx_remainder for async framer

[dbdbc]

Added an async server example.

The newly added constructor was required because read_header already reads from the stream before the websocket upgrade even happens.

Also, I fixed some stuff clippy was not satisfied with.

When looking at the client_async example, I noticed that the same buffer is used for rx and tx. While this might be fine in this simple case, I think in more general scenarios it could violate the contract described in framer_async::Framer::read

NOTE: any unused bytes read from the stream but not decoded are stored at the end
of the buffer to be used next time this read function is called. This also applies to
any unused bytes read when the connect handshake was made. Therefore it is important that
the caller does not clear this buffer between calls or use it for anthing other than reads.

This should in my opinion not be in an example in this way, but to have similar implementations for client_async and server_async, I also reused the same buffer for reading and writing.

Are there ways to enforce the mentioned contract? E. g. by not only storing indices into the buffer (frame_cursor, rx_remainder_len), but also a &mut [u8] in the Framer struct? Or would this conflict with the design you have in mind?

Anyway, that is not really part of this PR.

Thank you for creating this repo, and have a nice day!

[ninjasource]

Hi, thanks for taking the time to bring this issue to my attention and propose a solution. I'm going to play around with your changes to see if I like them. Very good point about reuse of that buffer for reads and writes, I'll have to give it some thought.