async-std TcpStream impls BufRead w/ AsyncWrite

[pinkforest]

eed4f0b

So wrapping TcpStream with BufReader should work ?

Even if I wrap it to BufReader then it wants AsyncWrite too complicating setting stuff like async-native-tls up ?

I could implement my own type with those implemented but probably doc neeeds to be adjusted - it's not as straightforward now in combination with the types it's usually used as the doc points to direct use with TcpStream.

45 |                 let mut transport = SmtpTransport::new(client, reader).await?;                
   |                                     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ the trait `AsyncWrite` is not implemented for `std::io::BufReader<TlsStream<async_std::net::TcpStream>>`

Repro: https://github.com/pinkforest/repro-async-std-async-smtp

For now I'm just using 0.8 as but got to refactor and port this stuff over off async-std probably

[link2xt]

Delta Chat uses tokio, but implements a wrapper type there because it also needs to progagate read timeouts over TLS and SOCKS5: https://github.com/deltachat/deltachat-core-rust/blob/a198e9fce84deb777c712a41d370ab8cf11cc240/src/net/session.rs

[link2xt]

You can use https://docs.rs/futures/0.3.31/futures/io/struct.BufReader.html, it passes AsyncWrite through:
https://docs.rs/futures/0.3.31/futures/io/struct.BufReader.html#impl-AsyncWrite-for-BufReader%3CR%3E