Skip to content

Commit 02a3ef3

Browse files
committed
fix: use direct clickhouse urls instead of tunneled urls (#2150)
<!-- Please make sure there is an issue that this PR is correlated to. --> ## Changes <!-- If there are frontend changes, please include screenshots. -->
1 parent 4c41e02 commit 02a3ef3

File tree

1 file changed

+11
-7
lines changed
  • packages/core/services/cluster/src/workflows/server/install/install_scripts/components/rivet

1 file changed

+11
-7
lines changed

packages/core/services/cluster/src/workflows/server/install/install_scripts/components/rivet/worker.rs

+11-7
Original file line numberDiff line numberDiff line change
@@ -94,13 +94,17 @@ pub fn configure(config: &rivet_config::Config) -> GlobalResult<String> {
9494
},
9595
},
9696
clickhouse: server_config.clickhouse.as_ref().map(|clickhouse| GlobalResult::Ok(ClickHouse {
97-
http_url: Url::parse(&format!(
98-
"https://127.0.0.1:{TUNNEL_CLICKHOUSE_PORT}",
99-
))?,
100-
native_url: Url::parse(&format!(
101-
"clickhouse://127.0.0.1:{TUNNEL_CLICKHOUSE_NATIVE_PORT}",
102-
))?,
103-
secure: false,
97+
// TODO: This doesn't work over Traefik since the client needs to specify the same
98+
// hostname as ClickHouse
99+
// http_url: Url::parse(&format!(
100+
// "https://127.0.0.1:{TUNNEL_CLICKHOUSE_PORT}",
101+
// ))?,
102+
// native_url: Url::parse(&format!(
103+
// "clickhouse://127.0.0.1:{TUNNEL_CLICKHOUSE_NATIVE_PORT}",
104+
// ))?,
105+
http_url: clickhouse.http_url.clone(),
106+
native_url: clickhouse.native_url.clone(),
107+
secure: clickhouse.secure,
104108
provision_users: HashMap::new(),
105109
..clickhouse.clone()
106110
})).transpose()?,

0 commit comments

Comments
 (0)