Skip to content

Commit d5755b3

Browse files
committed
Use CertifiedKey::new_unchecked
1 parent 01807f9 commit d5755b3

File tree

3 files changed

+4
-8
lines changed

3 files changed

+4
-8
lines changed

examples/client.rs

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -56,9 +56,7 @@ impl ResolvesClientCert for ClientCertResolver {
5656
if let Some(ref pin) = self.pin {
5757
signing_key.key().set_pin(pin).ok()?;
5858
}
59-
CertifiedKey::new(chain.into(), Box::new(signing_key))
60-
.ok()?
61-
.signer(sigschemes)
59+
CertifiedKey::new_unchecked(chain.into(), Box::new(signing_key)).signer(sigschemes)
6260
}
6361

6462
fn has_certs(&self) -> bool {

examples/server.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -83,7 +83,7 @@ impl ResolvesServerCert for ServerCertResolver {
8383
.map_err(|_| rustls::Error::NoSuitableCertificate)?;
8484
let certs = chain.into_iter().map(Into::into).collect();
8585

86-
CertifiedKey::new(certs, Box::new(key))?
86+
CertifiedKey::new_unchecked(certs, Box::new(key))
8787
.signer(client_hello.signature_schemes())
8888
.ok_or_else(|| rustls::Error::General("No common schemes".to_owned()))
8989
}

tests/test_client_server.rs

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -47,9 +47,7 @@ mod client {
4747
sigschemes: &[SignatureScheme],
4848
) -> Option<CertifiedSigner> {
4949
let (chain, signing_key) = get_chain(&self.0, &self.1).ok()?;
50-
CertifiedKey::new(chain.into(), Box::new(signing_key))
51-
.ok()?
52-
.signer(sigschemes)
50+
CertifiedKey::new_unchecked(chain.into(), Box::new(signing_key)).signer(sigschemes)
5351
}
5452

5553
fn has_certs(&self) -> bool {
@@ -133,7 +131,7 @@ mod server {
133131
.map_err(|_| rustls::Error::NoSuitableCertificate)?;
134132
let certs = chain.into_iter().map(Into::into).collect();
135133

136-
CertifiedKey::new(certs, Box::new(key))?
134+
CertifiedKey::new_unchecked(certs, Box::new(key))
137135
.signer(client_hello.signature_schemes())
138136
.ok_or_else(|| rustls::Error::General("No common schemes".to_owned()))
139137
}

0 commit comments

Comments
 (0)