cube-js/cube

Postgres connection - add option to set user "role"

Open

#8,530 创建于 2024年7月30日

在 GitHub 查看
 (2 评论) (0 反应) (0 负责人)Rust (19,563 star) (1,965 fork)batch import
data source driverdriver:postgreshelp wanted

描述

Is your feature request related to a problem? Please describe. I am trying to connect my Postgres instance to Cube Core - however our instance is set in a way where in addition to providing Username and Password we also need to provide a user role (https://www.postgresql.org/docs/current/sql-set-role.html)

Based on my research seems like Cube currently does not support this.

It would be great if we could support this as an optional configuration option just like dbt (https://docs.getdbt.com/docs/core/connect-data-platform/postgres-setup)

Describe the solution you'd like It would be great if we could have an environment variable to set user role, just like the ones we have for username and password. Also, @igorlukanin suggested another option on slack that - we could achieve this via "initSql" config option, just like for it is done in DuckDB https://github.com/cube-js/cube/blob/34d319a8d81744bde1e655fefeb23496f2fe1089/packages/cubejs-duckdb-driver/src/DuckDBDriver.ts#L163-L173

Thanks

贡献者指南

技术栈
postgresqlnodejs
领域
backenddatabase
议题类型
feature
难度面向新贡献者的预计实现难度，1 表示很小改动，5 表示专家级工作。
3
预计时间有经验贡献者完成调查、实现、测试并准备 pull request 的粗略时间范围。
half day
活动状态议题当前的可参与程度：新鲜、活跃、陈旧、阻塞或等待维护者输入。
fresh
清晰度议题是否清楚说明期望改动、验收标准和下一步。
mostly clear
前置要求
Basic understanding of Postgres connection settingsFamiliarity with Cube's driver architecture
新手友好度1-100 的估计分数，表示该议题对首次贡献者的友好程度。
60
研究方向
The issue requests adding an optional 'role' configuration for Postgres connections. A possible approach is to introduce an environment variable (e.g., CUBEJS DB ROLE) similar to existing username/password variables, or to implement it via the 'initSql' option as done for DuckDB in packages/cubejs duckdb driver/src/DuckDBDriver.ts (lines 163-173). To proceed, examine the Postgres driver code (likely in packages/cubejs postgres driver) to understand how connection parameters are passed, and then add the role parameter to the connection configuration. The maintainer suggested the initSql route, so that might be the simpler path. No linked PRs or assignees indicate this is open for contribution.