Take named options in `PostgreSqlContainer.withSSL`
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
- 難易度
- 2/5
- 見積もり時間
- 1〜3時間
- 初心者へのやさしさ
- 74/100
- issue の種類
- 機能追加
- 明瞭さ
- 明確に書かれている
- 活発さ
- 活発
- 技術スタック
- docker, postgresql, typescript
調査の方向性
packages/modules/postgresql/src/postgresql-container.tsから始めて、PostgreSqlContainer.withSSLをwithSSLCertおよび#1518と調整された位置引数のオーバーロードと併せて確認してください。名前付きフィールド形式を追加し、位置引数形式は非推奨のオーバーロードとして維持してください。呼び出し側が{ cert, key, ca }を渡せ、既存の位置引数シグネチャが非推奨のオーバーロードとして引き続き利用できれば完了です。
索引モデルが issue の本文から書いたものです。
説明
What happens
PostgreSqlContainer has two SSL methods that take the same three file paths in different positions:
withSSL(certFile: string, keyFile: string, caCertFile?: string)
withSSLCert(caCertFile: string, certFile: string, keyFile: string)
All three are plain strings, so passing them in the wrong order compiles. The mistake shows up only when the server fails to start or a client fails to verify.
Expected
A form with named fields, for example withSSL({ cert, key, ca }), with the positional forms kept as deprecated overloads.
Where
packages/modules/postgresql/src/postgresql-container.ts.
Notes
withSSLCert(caCertFile, certFile, keyFile)matches Go'spostgres.WithSSLCert(caCertFile, certFile, keyFile).withSSLis the one that differs.- Both were added in #1224.
- The same applies to any module method that takes several paths or credentials positionally. Worth a line in the
add-moduleskill.
Dependencies
- Nothing blocks this. It can start now.
- Coordinate with #1518: for the positional overloads.
- 主要言語
- TypeScript
- スター
- 2.6k
- フォーク
- 274
- 平均マージ
- 1日 14時間
- マージ済み PR（30日）
- 44
環境構築
このプロジェクトの開発コンテナを、あなたの GitHub アカウントでブラウザ上に起動します。
- Dockerfile または Docker Compose ファイルあり
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
testcontainers/testcontainers-node のほかの issue
-
enhancement minor
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
testcontainers/testcontainers-node#1533 ·
メンテナーはふだん 1 日以内に返信
-
enhancement minor
難易度 2/5 1〜3時間 初心者へのやさしさ 77/100
testcontainers/testcontainers-node#1531 ·
メンテナーはふだん 1 日以内に返信
-
enhancement minor
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
testcontainers/testcontainers-node#1529 ·
メンテナーはふだん 1 日以内に返信
-
enhancement minor
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
testcontainers/testcontainers-node#1528 ·
メンテナーはふだん 1 日以内に返信
-
bug patch
難易度 2/5 1〜3時間 初心者へのやさしさ 79/100
testcontainers/testcontainers-node#1526 ·
メンテナーはふだん 1 日以内に返信
testcontainers/testcontainers-node の issue をすべて見る
似ている issue
-
難易度 1/5 1時間未満 初心者へのやさしさ 85/100
-
area:cowork area:desktop documentation enhancement platform:macos
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
anthropics/claude-code#101720 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 77/100
apache/rocketmq-dashboard#6610 ·
メンテナーはふだん 3 日以内に返信
-
Add: MS NOWオープンcheck:passed streams:add
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
メンテナーはふだん 1 日以内に返信
-
Global update drops overrideAccess, so Local API updateGlobal fails filterOptions validation (3.90 regression)対応中かも @PINYOPATTANAWASANPORN が今日担当しました。 オープンarea: core Bug
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
payloadcms/payload#18674 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信