TrinoContainer.get_connection_url() returns the internal port, not the mapped host port
还没有人认领这个 Issue。
评估
- 难度
- 2/5
- 预计耗时
- 1-3 小时
- 新手友好度
- 78/100
调研方向
定位 TrinoContainer 的实现,并阅读 get_connection_url()、get_exposed_port() 以及可比较的 CockroachDBContainer 或 CrateDBContainer URL 方法。为映射端口的 URL 行为添加覆盖,并验证通过 get_connection_url() 创建的连接能够访问正在运行的 Trino 容器。
由索引模型根据 Issue 内容生成。
描述
TrinoContainer.get_connection_url() builds the URL as trino://{user}@{host}:{self.port}, and self.port is the fixed container-internal port set in __init__ (8080 by default). It's never resolved through get_exposed_port(). Since Docker maps that to a random host port, the URL this method returns can't actually connect to anything.
Repro (4.15.0):
from testcontainers.community.trino import TrinoContainer
from sqlalchemy import create_engine, text
with TrinoContainer() as trino:
print(trino.port) # 8080
print(trino.get_exposed_port(trino.port)) # the real mapped port, e.g. 58092
print(trino.get_connection_url()) # trino://test@localhost:8080 - wrong
engine = create_engine(trino.get_connection_url())
with engine.connect() as conn:
conn.execute(text("select 1")) # ConnectionRefusedError / TrinoConnectionError
Building the URL manually with get_exposed_port() instead of self.port connects fine.
For comparison, CockroachDBContainer/CrateDBContainer don't have this problem since they route through _create_connection_url(), which does resolve the exposed port. Looks like an isolated miss in the Trino module rather than a pattern elsewhere.
Suggested fix, matching how the other modules do it:
def get_connection_url(self):
return f"trino://{self.user}@{self.get_container_host_ip()}:{self.get_exposed_port(self.port)}"
Happy to open a PR with this if useful. Found it while wiring up a nightly testcontainers job for apache/superset's db engine spec tests.
- 主要语言
- Python
- 星标
- 2.3k
- 派生
- 386
- 平均合并
- 4 小时 40 分钟
- 30 天内合并 PR
- 1
贡献指南
从这里开始
- 先读完整个 Issue,再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库,在一个分支上完成修改。
- 提交 Pull Request,并在描述里引用这个 Issue 编号。
testcontainers/testcontainers-python 的其他 Issue
-
难度 2/5 1-3 小时 新手友好度 88/100
testcontainers/testcontainers-python#1115 · 1 个 reaction ·
-
难度 2/5 1-3 小时 新手友好度 67/100
testcontainers/testcontainers-python#1086 · 1 条评论 ·
-
难度 2/5 1-3 小时 新手友好度 68/100
testcontainers/testcontainers-python#578 · 5 条评论 ·
-
难度 3/5 1-2 天 新手友好度 65/100
-
难度 3/5 1-2 天 新手友好度 68/100
查看 testcontainers/testcontainers-python 的全部 Issue
相似的 Issue
-
area: harness bug status: needs-triage
难度 2/5 1-3 小时 新手友好度 75/100
Human-Agent-Society/reef#625 ·
-
难度 2/5 1-3 小时 新手友好度 70/100
-
难度 1/5 1 小时以内 新手友好度 80/100
learningequality/kolibri#15351 · 2 条评论 ·
-
难度 2/5 1-3 小时 新手友好度 75/100
-
Name consistency 未关闭
难度 2/5 1-3 小时 新手友好度 75/100
eellak/triplestore#65 · 1 条评论 ·