Hacktoberfest 2026: những issue maintainer đã đánh dấu cho tháng Mười, đang mở và phù hợp người mới. Xem issue Hacktoberfest

Wire TlsConfig into ConnectionBuilder and Config

Đang mở
  • Có gì không ổn?
#66 0 bình luận 0 reaction 0 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 1 ngày

Chưa có ai nhận issue này.

Đánh giá

Độ khó
4/5
Thời gian dự kiến
3-5 ngày
Mức phù hợp với người mới
52/100
Loại issue
Tính năng
Độ rõ ràng
Khá rõ ràng
Mức độ hoạt động
Ít trao đổi
Công nghệ
rust
Lĩnh vực
backend-api-design, security

Hướng nghiên cứu

Start by reading hyperdb-api-core/src/config.rs and hyperdb-api/src/connection_builder.rs, then trace the existing Client::connect call and lower-layer TlsConfig usage. Run the existing TLS tests before adding high-level integration coverage; done means TLS is exposed through the sync and async builders, demonstrated in hyperdb-api/examples/, and documented on ConnectionBuilder.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

enhancement

Summary

TlsConfig exists in hyperdb-api-core but is not reachable through the public Connection / AsyncConnection / ConnectionBuilder surface. There is no with_tls() / tls() method on any of them. TLS code is built and tested at the lower layer but has no on-ramp from the high-level API.

Current state

$ grep -rn "fn with_tls\|fn tls\|TlsConfig\|tls_config" \
    hyperdb-api/src/connection_builder.rs \
    hyperdb-api-core/src/config.rs
# (no output)

The original gap analysis (§7 of docs/RUST_API_GAP_ANALYSIS.md in the predecessor repo) called this out:

TlsConfig exists with a full builder. But the module doc says "TODO: with_tls not yet on Config" — i.e. there is no way to pass a TlsConfig to Client::connect(&Config). Config has no tls field/method. TLS code exists but isn't reachable through the public Connection/AsyncConnection or ConnectionBuilder.

Proposed work

  • Add a tls(TlsConfig) -> Self method on Config in hyperdb-api-core (or whichever struct Connection consumes for its TLS settings).
  • Add a tls(TlsConfig) -> Self method on ConnectionBuilder and its async counterparts.
  • Wire the field through to the actual Client::connect call so it has effect.
  • Add an integration test that connects to a TLS-enabled Hyper endpoint via the high-level API.
  • Add an example under hyperdb-api/examples/ demonstrating TLS configuration.
  • Update the rustdoc on ConnectionBuilder to mention TLS.

Backwards compatibility

Purely additive — new methods, no changes to existing signatures.

Note on cert verification

Whatever default TlsConfig produces today should remain the default. This issue is about exposing TLS, not redesigning the security defaults.

Ngôn ngữ chính
Rust
Star
3
Fork
2
Merge trung bình
23 giờ 11 phút
Pull request đã merge (30 ngày)
65

Chuẩn bị môi trường

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của tableau/hyper-api-rust

Tất cả issue của tableau/hyper-api-rust

Issue tương tự

Thêm issue về Rust

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.