Skip to content

Commit a109491

Browse files
authored
Always respect EDGEDB_CLOUD_PROFILE (geldata#235)
Cloud instances, unlike local instances, do not represent local configuration, so `EDGEDB_CLOUD_PROFILE` should always be respected when connecting to such instances regardless of how compound connection options have been specified.
1 parent 9ba86e8 commit a109491

File tree

1 file changed

+9
-8
lines changed

1 file changed

+9
-8
lines changed

edgedb-tokio/src/builder.rs

Lines changed: 9 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1046,13 +1046,10 @@ impl Builder {
10461046
}
10471047
}
10481048
}
1049-
async fn preliminary_env(&self, cfg: &mut ConfigInner,
1049+
1050+
async fn secret_key_env(&self, cfg: &mut ConfigInner,
10501051
errors: &mut Vec<Error>)
10511052
{
1052-
cfg.cloud_profile = self.cloud_profile.clone().or_else(|| {
1053-
get_env("EDGEDB_CLOUD_PROFILE")
1054-
.map_err(|e| errors.push(e)).ok().flatten()
1055-
});
10561053
cfg.secret_key = self.secret_key.clone().or_else(|| {
10571054
get_env("EDGEDB_SECRET_KEY")
10581055
.map_err(|e| errors.push(e)).ok().flatten()
@@ -1334,6 +1331,11 @@ impl Builder {
13341331
verifier: Arc::new(tls::NullVerifier),
13351332
};
13361333

1334+
cfg.cloud_profile = self.cloud_profile.clone().or_else(|| {
1335+
get_env("EDGEDB_CLOUD_PROFILE")
1336+
.map_err(|e| errors.push(e)).ok().flatten()
1337+
});
1338+
13371339
let complete = if self.host.is_some() ||
13381340
self.port.is_some() ||
13391341
self.unix_path.is_some() ||
@@ -1343,20 +1345,19 @@ impl Builder {
13431345
self.credentials_file.is_some()
13441346
{
13451347
cfg.secret_key = self.secret_key.clone();
1346-
cfg.cloud_profile = self.cloud_profile.clone();
13471348
self.compound_owned(&mut cfg, &mut errors).await;
13481349
self.granular_owned(&mut cfg, &mut errors).await;
13491350
true
13501351
} else if
13511352
COMPOUND_ENV_VARS.iter().any(|x| env::var_os(x).is_some()) ||
13521353
has_port_env()
13531354
{
1354-
self.preliminary_env(&mut cfg, &mut errors).await;
1355+
self.secret_key_env(&mut cfg, &mut errors).await;
13551356
self.compound_env(&mut cfg, &mut errors).await;
13561357
self.granular_env(&mut cfg, &mut errors).await;
13571358
true
13581359
} else {
1359-
self.preliminary_env(&mut cfg, &mut errors).await;
1360+
self.secret_key_env(&mut cfg, &mut errors).await;
13601361
let complete = self.read_project(&mut cfg, &mut errors).await;
13611362
self.granular_env(&mut cfg, &mut errors).await;
13621363
complete

0 commit comments

Comments
 (0)