Skip to content

fix: Configured keystores must contain at least one key entry#2857

Merged
dadrus merged 2 commits intoreleasefrom
fix/panic_on_invalid_keystore
Nov 6, 2025
Merged

fix: Configured keystores must contain at least one key entry#2857
dadrus merged 2 commits intoreleasefrom
fix/panic_on_invalid_keystore

Conversation

@dadrus
Copy link
Copy Markdown
Owner

@dadrus dadrus commented Nov 6, 2025

Related issue(s)

fixes #2833

Checklist

  • I agree to follow this project's Code of Conduct.
  • I have read, and I am following this repository's Contributing Guidelines.
  • I have read the Security Policy.
  • I have referenced an issue describing the bug/feature request.
  • I have added tests that prove the correctness of my implementation.

Description

This PR implements a check to ensure that configured keystores contain at least one key entry

@codecov
Copy link
Copy Markdown

codecov bot commented Nov 6, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 91.29%. Comparing base (b68e414) to head (210e510).
⚠️ Report is 2 commits behind head on release.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@             Coverage Diff             @@
##           release    #2857      +/-   ##
===========================================
+ Coverage    91.23%   91.29%   +0.05%     
===========================================
  Files          283      283              
  Lines        10667    10670       +3     
===========================================
+ Hits          9732     9741       +9     
+ Misses         680      677       -3     
+ Partials       255      252       -3     

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

@dadrus dadrus merged commit e1a4c27 into release Nov 6, 2025
39 of 41 checks passed
@dadrus dadrus deleted the fix/panic_on_invalid_keystore branch November 6, 2025 09:47
@dadrus dadrus linked an issue Nov 6, 2025 that may be closed by this pull request
3 tasks
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.

Using keystores without a private key entry for tls results in panic

1 participant