search

steffengy / schannel-rs

Schannel API-bindings for rust (provides an interface for native SSL/TLS using windows APIs)
MIT License
49 stars 51 forks source link

restore BC: the validation callback (+TlsStream) should be Sync/Send,… #30

Closed steffengy closed 7 years ago

steffengy commented 7 years ago

Just fixing some missed details.

sfackler commented 7 years ago

Looks good to me. Might be worth adding in some little checks to keep this from regressing in the future:

fn _is_sync() {
    fn sync<T: Sync>() {}
    sync::<TlsStream<()>>();
}
sstecko commented 7 years ago

hah. Just noticed this when trying to update rust-native-tls. Thanks