async-rs / async-tls

A TLS implementation over AsyncRead and AsyncWrite
https://async.rs
Apache License 2.0
165 stars 47 forks source link

update futures dep to 0.3 #6

Closed serzhiio closed 5 years ago

dignifiedquire commented 5 years ago
skade commented 5 years ago

I'm tracking and will do the update later.

skade commented 5 years ago

Fixed the tests in https://github.com/async-rs/async-tls/commit/cd44fd7adfc24632c07a941c2daf4e05dc68be9e, but I'm not fully happy, especially here:

https://github.com/async-rs/async-tls/blob/master/tests/test.rs#L40-L46

dignifiedquire commented 5 years ago

Yeah that is not very nice, but given the current construction we can't implement AsyncRead for &TlsStream :/

Maybe @stjepang has an idea on how we could improve on this?