Fix TLS infinite busy loop when write/read handlers are removed #3508
Codecov / codecov/project
succeeded
Apr 15, 2026 in 0s
76.36% (-0.03%) compared to c4db208
View this Pull Request on Codecov
76.36% (-0.03%) compared to c4db208
Details
Codecov Report
❌ Patch coverage is 58.13953% with 18 lines in your changes missing coverage. Please review.
✅ Project coverage is 76.36%. Comparing base (c4db208) to head (7fd39fd).
| Files with missing lines | Patch % | Lines |
|---|---|---|
| src/unit/test_tls.cpp | 54.28% | 16 Missing |
| src/valkey-cli.c | 75.00% | 2 Missing |
Additional details and impacted files
@@ Coverage Diff @@
## unstable #3508 +/- ##
============================================
- Coverage 76.38% 76.36% -0.03%
============================================
Files 159 160 +1
Lines 79809 79850 +41
============================================
+ Hits 60966 60975 +9
- Misses 18843 18875 +32 | Files with missing lines | Coverage Δ | |
|---|---|---|
| src/tls.c | 17.64% <ø> (ø) |
|
| src/valkey-cli.c | 59.91% <75.00%> (+<0.01%) |
⬆️ |
| src/unit/test_tls.cpp | 54.28% <54.28%> (ø) |
🚀 New features to boost your workflow:
- ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
- 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.
Loading