Add mTLS support to Milvus S3 client#48882
Add mTLS support to Milvus S3 client#48882junjieqi wants to merge 2 commits intomilvus-io:masterfrom
Conversation
configuration, enabling mTLS authentication with S3-compatible endpoints that require client certificates . The client cert/key are loaded via tls.LoadX509KeyPair and injected into the HTTP transport's TLS config.
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: junjieqi The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
|
Welcome @junjieqi! It looks like this is your first PR to milvus-io/milvus 🎉 |
|
@junjieqi Thanks for your contribution. Please submit with DCO, see the contributing guide https://github.com/milvus-io/milvus/blob/master/CONTRIBUTING.md#developer-certificate-of-origin-dco. |
|
Invalid PR Title Format Detected Your PR submission does not adhere to our required standards. To ensure clarity and consistency, please meet the following criteria:
Required Title Structure: Where Example: Please review and update your PR to comply with these guidelines. |
|
[ci-v2-notice] To rerun ci-v2 checks, comment with:
If you have any questions or requests, please contact @zhikunyao. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #48882 +/- ##
==========================================
- Coverage 84.44% 83.66% -0.79%
==========================================
Files 642 654 +12
Lines 101106 103173 +2067
==========================================
+ Hits 85382 86321 +939
- Misses 15724 16800 +1076
- Partials 0 52 +52
🚀 New features to boost your workflow:
|
No description provided.