search

async-rs / async-tls

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

Unify API #24

Open sehz opened 4 years ago

sehz commented 4 years ago

In the async-native-tls, there is a single TlsStream, but in this crate, there are two different Stream implementation. It would be nice if we have unified API that can support different engine.

skade commented 4 years ago

I'm trying to remember what the exact cause for having both streams was. I tried unifying both of them a while back, but if I remember right, it's at least not trivial, at least without dynamic dispatch.

That said, I'd prefer one TLS implementation for all runtimes.