Skip to content

Commit

Permalink
fix: Allow config http protocol to be passed to clickhouse driver. (#…
Browse files Browse the repository at this point in the history
…9195)

Co-authored-by: Konstantin Burkalev <[email protected]>
  • Loading branch information
BrandonFreeman-Cavallo and KSDaemon authored Mar 10, 2025
1 parent 29f80c9 commit 0c15b01
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion packages/cubejs-clickhouse-driver/src/ClickHouseDriver.ts
Original file line number Diff line number Diff line change
Expand Up @@ -142,7 +142,7 @@ export class ClickHouseDriver extends BaseDriver implements DriverInterface {
const dataSource = config.dataSource ?? assertDataSource('default');
const host = config.host ?? getEnv('dbHost', { dataSource });
const port = config.port ?? getEnv('dbPort', { dataSource }) ?? 8123;
const protocol = config.protocol ?? getEnv('dbSsl', { dataSource }) ? 'https:' : 'http:';
const protocol = config.protocol ?? (getEnv('dbSsl', { dataSource }) ? 'https:' : 'http:');
const url = `${protocol}//${host}:${port}`;

const username = config.username ?? getEnv('dbUser', { dataSource });
Expand Down

0 comments on commit 0c15b01

Please sign in to comment.