Skip to content

Bugfix: Send the TLS alert to the peer upon a ssl.CertificateError. #3413

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
sasozivanovic wants to merge 8 commits into
base: main
Choose a base branch
from
+162 −18

Another attempt to fix type checking.

71fe831
Select commit
Loading
Failed to load commit list.
Open

Bugfix: Send the TLS alert to the peer upon a ssl.CertificateError. #3413

Another attempt to fix type checking.
71fe831
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Apr 21, 2026 in 0s

100.00000% (target 100.00000%)

View this Pull Request on Codecov

100.00000% (target 100.00000%)

Details

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00000%. Comparing base (82af3ab) to head (71fe831).
⚠️ Report is 7 commits behind head on main.

Additional details and impacted files 
@@               Coverage Diff               @@
##                 main        #3413   +/-   ##
===============================================
  Coverage   100.00000%   100.00000%           
===============================================
  Files             128          128           
  Lines           19417        19482   +65     
  Branches         1318         1321    +3     
===============================================
+ Hits            19417        19482   +65
Files with missing lines Coverage Δ
src/trio/_ssl.py 100.00000% <100.00000%> (ø)
src/trio/_tests/test_ssl.py 100.00000% <100.00000%> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
View more details on Codecov