Take named options in `PostgreSqlContainer.withSSL`
维护者通常 1 天内回复
还没有人认领这个 Issue。
评估
- 难度
- 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 小时
- 30 天内合并 PR
- 44
环境准备
在浏览器里用你自己的 GitHub 账号启动这个项目的开发容器。
- 提供 Dockerfile 或 Docker Compose 文件
- 没有 Pull Request 模板
- 阅读贡献指南
从这里开始
- 先读完整个 Issue，再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库，在一个分支上完成修改。
- 提交 Pull Request，并在描述里引用这个 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
-
难度 2/5 1-3 小时 新手友好度 82/100
维护者通常 1 天内回复
-
high priority
难度 2/5 1-3 小时 新手友好度 62/100
CI-CMG/deep-sea-corals-exb#84 · 1 条评论 ·
-
Create/import wallet form shows raw "password must be at least 1 characters" instead of the translated message可能已有人在做 @CapThunder19 今天认领。 未关闭bug
难度 2/5 1-3 小时 新手友好度 82/100
joinmarket-webui/jam#1538 ·
维护者通常 1 天内回复
-
t0ggles
难度 2/5 1-3 小时 新手友好度 70/100
nolimits4web/swiper#8240 · 1 条评论 ·
-
难度 2/5 1-3 小时 新手友好度 82/100
维护者通常 1 天内回复