blackbeam / mysql_async

Asyncronous Rust Mysql driver based on Tokio.
Apache License 2.0
372 stars 113 forks source link

Update rustls #304

Open serprex opened 2 months ago

serprex commented 2 months ago

rustls replaced ring with aws-lc-rs as default crypto backend, expose features to select between the two, along with a feature on whether to enable tls 1.2

serprex commented 2 months ago

as an aside, rust features are supposed to be additive, this is not currently the case with rustls/native-tls features. If both features are enabled the compile fails. Would you be interested in having changes made to rectify that?

I think the fix would be to have each crypto backend use a different Opts (ie, ssl_opts would be ssl_opts_native & ssl_opts_rustls). Could get more complicated with traits, but doesn't seem necessary