Wire TlsConfig into ConnectionBuilder and Config
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
- 難易度
- 4/5
- 見積もり時間
- 3〜5日
- 初心者へのやさしさ
- 52/100
- issue の種類
- 機能追加
- 明瞭さ
- おおむね明確
- 活発さ
- 静か
- 技術スタック
- rust
調査の方向性
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.
索引モデルが issue の本文から書いたものです。
説明
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:
TlsConfigexists with a full builder. But the module doc says "TODO: with_tls not yet on Config" — i.e. there is no way to pass aTlsConfigtoClient::connect(&Config).Confighas no tls field/method. TLS code exists but isn't reachable through the publicConnection/AsyncConnectionorConnectionBuilder.
Proposed work
- Add a
tls(TlsConfig) -> Selfmethod onConfiginhyperdb-api-core(or whichever structConnectionconsumes for its TLS settings). - Add a
tls(TlsConfig) -> Selfmethod onConnectionBuilderand its async counterparts. - Wire the field through to the actual
Client::connectcall 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
ConnectionBuilderto 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.
- 主要言語
- Rust
- スター
- 3
- フォーク
- 2
- 平均マージ
- 23時間 11分
- マージ済み PR(30日)
- 65
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
tableau/hyper-api-rust のほかの issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
tableau/hyper-api-rust#294 ·
メンテナーはふだん 1 日以内に返信
-
難易度 4/5 3〜5日 初心者へのやさしさ 35/100
tableau/hyper-api-rust#311 ·
メンテナーはふだん 1 日以内に返信
-
難易度 4/5 3〜5日 初心者へのやさしさ 45/100
tableau/hyper-api-rust#305 ·
メンテナーはふだん 1 日以内に返信
-
Windows Named Pipe: verify DACL denies other users, and measure read-path perf for MCP workloadsオープン
難易度 4/5 3〜5日 初心者へのやさしさ 38/100
tableau/hyper-api-rust#302 ·
メンテナーはふだん 1 日以内に返信
-
難易度 3/5 1〜2日 初心者へのやさしさ 72/100
tableau/hyper-api-rust#300 ·
メンテナーはふだん 1 日以内に返信
tableau/hyper-api-rust の issue をすべて見る
似ている issue
-
Progress difficulty filter lists Hard before Medium対応中かも @Pandamachi が今日担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 86/100
sysprog21/codetrial#281 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
C-bug
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
rust-lang/rust-analyzer#23501 ·
メンテナーはふだん 1 日以内に返信
-
Streamable HTTP client: a 401 or 403 with a JSON-RPC error body and no WWW-Authenticate loses its HTTP status対応中かも このイシューにリンクされたプルリクエストがオープン中、またはマージ済みです。 オープンbug P2 ready for work T-security T-transport
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
modelcontextprotocol/rust-sdk#1339 ·
メンテナーはふだん 3 日以内に返信
-
scripts/gen-gallery.py:118: a ready session now reports in_progress, so SESSION_READY_OLD can goオープンnightly-audit
難易度 2/5 1〜3時間 初心者へのやさしさ 84/100
antithesishq/snouty#396 ·
メンテナーはふだん 1 日以内に返信
-
French BIP39 wordlist starts with a UTF-8 BOM, so generated French mnemonics carry U+FEFF and derive a non-canonical seed対応中かも @Kshot3000 が今日担当しました。 オープン
難易度 1/5 1時間未満 初心者へのやさしさ 91/100
ergoplatform/sigma-rust#976 ·
メンテナーはふだん 1 日以内に返信