Merge pull request #776 from squidowl/dont-panic-on-invalid-certs

Ignore errors while adding der certificates
squidowl · Feb 15, 2025 · df9dcc0 · df9dcc0
2 parents 5857467 + db96a09
commit df9dcc0
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -10,6 +10,7 @@ Fixed:
 - [`infinite_scroll`](https://halloy.squidowl.org/configuration/buffer.html?highlight=infinite#infinite_scroll) was defaulting to `false`, contrary to its documented default value.  Now defaults to `true`
 - A rare issue where the app opens smaller than when closed
 - Closing the application while it is minimized (on Windows) causes it to open with the wrong size and position next time
+- TLS connection issue on Windows (10+)
 
 # 2025.1 (2025-02-02)
 

diff --git a/irc/src/connection/tls.rs b/irc/src/connection/tls.rs
@@ -30,7 +30,7 @@ pub async fn connect<'a>(
         let mut roots = rustls::RootCertStore::empty();
 
         for cert in rustls_native_certs::load_native_certs()? {
-            roots.add(cert).unwrap();
+            let _ = roots.add(cert);
         }
 
         if let Some(cert_path) = root_cert_path {