https://github.com/Tencent/tquic/issues/153#452
https://github.com/Tencent/tquic/issues/153#452tiemuhua wants to merge 2 commits intoTencent:developfrom
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #452 +/- ##
===========================================
- Coverage 95.36% 95.28% -0.09%
===========================================
Files 50 50
Lines 34238 34273 +35
===========================================
+ Hits 32651 32656 +5
- Misses 1587 1617 +30 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Pull Request Overview
This PR introduces mutual authentication configurations for both client and server TLS setups while slightly improving error handling paths.
- Adds a new function for mutual authentication client configuration.
- Adds a new function for mutual authentication server configuration.
- Slightly refactors variable typing and error handling in existing functions.
| ) -> Result<Self> { | ||
| if cert_file.is_empty() || key_file.is_empty() { | ||
| return Err(Error::TlsFail(format!( | ||
| "cert_file({:?}) key_file({:?})", |
There was a problem hiding this comment.
Consider improving the error message here to clearly state that both certificate and key files are required, e.g., 'Certificate file and key file must be provided, but received cert_file: {:?}, key_file: {:?}'.
| "cert_file({:?}) key_file({:?})", | |
| "Certificate file and key file must be provided, but received cert_file: {:?}, key_file: {:?}", |
| Ok(tls_config) | ||
| } | ||
|
|
||
| pub fn new_mutual_authentication_client_config( |
There was a problem hiding this comment.
Add a doc comment for 'new_mutual_authentication_client_config' to explain its purpose and usage, aligning its documentation with that of 'new_server_config' for clarity.
No description provided.