Skip to content

Fix ratis TLS not working#16510

Merged
HTHou merged 4 commits into
masterfrom
fix_ratis_tls_config
Sep 29, 2025
Merged

Fix ratis TLS not working#16510
HTHou merged 4 commits into
masterfrom
fix_ratis_tls_config

Conversation

@HTHou

@HTHou HTHou commented Sep 28, 2025

Copy link
Copy Markdown
Contributor

Description

The current TLS configuration for Ratis is not correct.

@codecov

codecov Bot commented Sep 28, 2025

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 29.03226% with 22 lines in your changes missing coverage. Please review.
✅ Project coverage is 38.63%. Comparing base (254f016) to head (a3f6f1e).
⚠️ Report is 3 commits behind head on master.

Files with missing lines Patch % Lines
...atis/utils/NoHostnameVerificationTrustManager.java 0.00% 20 Missing ⚠️
.../org/apache/iotdb/consensus/ratis/utils/Utils.java 0.00% 2 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##             master   #16510      +/-   ##
============================================
- Coverage     38.64%   38.63%   -0.01%     
  Complexity      207      207              
============================================
  Files          4933     4934       +1     
  Lines        326172   326200      +28     
  Branches      41383    41383              
============================================
- Hits         126033   126016      -17     
- Misses       200139   200184      +45     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@sonarqubecloud

Copy link
Copy Markdown

@HTHou HTHou merged commit e03560f into master Sep 29, 2025
35 of 36 checks passed
@HTHou HTHou deleted the fix_ratis_tls_config branch September 29, 2025 01:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants