sander/rust-imap
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Update msrv

Browse source 
Avoid Arc::into_inner since it requires 1.70
This commit is contained in:
Jon Gjengset 2024-03-31 10:27:42 +02:00
parent 00fd45a199
commit e71fd87417
2 changed files with 5 additions and 8 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
.github/workflows/check.yml vendored
View file

@ -99,7 +99,7 @@ jobs:
# https://docs.github.com/en/actions/learn-github-actions/contexts#context-availability # https://docs.github.com/en/actions/learn-github-actions/contexts#context-availability
strategy: strategy:
matrix: matrix:
msrv: ["1.57.0"] # base64 0.21 requires 1.57 msrv: ["1.65.0"]
name: ubuntu / ${{ matrix.msrv }} name: ubuntu / ${{ matrix.msrv }}
steps: steps:
- uses: actions/checkout@v4 - uses: actions/checkout@v4

11
src/client_builder.rs
View file

@ -21,7 +21,7 @@ use std::sync::Arc;
#[cfg(feature = "rustls-tls")] #[cfg(feature = "rustls-tls")]
#[derive(Debug)] #[derive(Debug)]
struct NoCertVerification(rustls::client::WebPkiServerVerifier); struct NoCertVerification(Arc<rustls::client::WebPkiServerVerifier>);
#[cfg(feature = "rustls-tls")] #[cfg(feature = "rustls-tls")]
impl rustls::client::danger::ServerCertVerifier for NoCertVerification { impl rustls::client::danger::ServerCertVerifier for NoCertVerification {
@ -365,12 +365,9 @@ where
config config
.dangerous() .dangerous()
.set_certificate_verifier(Arc::new(NoCertVerification( .set_certificate_verifier(Arc::new(NoCertVerification(
Arc::into_inner( rustls::client::WebPkiServerVerifier::builder(Arc::new(CACERTS.clone()))
rustls::client::WebPkiServerVerifier::builder(Arc::new(CACERTS.clone())) .build()
.build() .expect("can construct standard verifier"),
.expect("can construct standard verifier"),
)
.expect("just constructed, so should only be one"),
))); )));
} }
let ssl_conn: RustlsConnector = config.into(); let ssl_conn: RustlsConnector = config.into();