205hlab-mirrors/authentik
2
0
Fork 0
mirror of https://github.com/goauthentik/authentik.git synced 2026-06-17 19:09:11 +03:00
Code Issues Packages Projects Releases Wiki Activity

packages/ak-common/config: fix boolean parsing from env variable (#21835)

Browse Source
This commit is contained in:
Marc 'risson' Schmitt
2026-04-27 12:53:47 +02:00
committed by GitHub
parent 97c9626bd4
commit 5c3cd2c6ed
1 changed files with 78 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
packages/ak-common/src/config/mod.rs
+78 -1
View File
@@ -78,7 +78,9 @@ impl Config {
builder = builder.add_source(
config_rs::Environment::with_prefix("AUTHENTIK")
.prefix_separator("_")
.separator("__"),
.separator("__")
.try_parsing(true)
.list_separator(","),
);
if let Some(overrides) = overrides {
builder = builder.add_source(config_rs::File::from_str(
@@ -455,4 +457,79 @@ mod tests {
super::set(json!({"secret_key": "my_new_secret_key"})).expect("failed to set config");
assert_eq!(super::get().secret_key, "my_new_secret_key");
}
#[test]
fn env_bool_true() {
#[expect(unsafe_code, reason = "testing")]
// SAFETY: testing
unsafe {
env::set_var("AUTHENTIK_DEBUG", "true");
}
let (config, _) = super::Config::load(&[], None).expect("failed to load config");
assert!(config.debug);
}
#[test]
fn env_bool_false() {
#[expect(unsafe_code, reason = "testing")]
// SAFETY: testing
unsafe {
env::set_var("AUTHENTIK_DEBUG", "false");
}
let (config, _) = super::Config::load(&[], None).expect("failed to load config");
assert!(!config.debug);
}
// See https://github.com/rust-cli/config-rs/issues/443
// #[test]
// fn env_list_empty() {
// #[expect(unsafe_code, reason = "testing")]
// // SAFETY: testing
// unsafe {
// env::set_var("AUTHENTIK_LISTEN__HTTP", "");
// }
//
// let (config, _) = super::Config::load(&[], None).expect("failed to load config");
//
// assert_eq!(config.listen.http, []);
// }
#[test]
fn env_list_one_element() {
#[expect(unsafe_code, reason = "testing")]
// SAFETY: testing
unsafe {
env::set_var("AUTHENTIK_LISTEN__HTTP", "[::1]:9000");
}
let (config, _) = super::Config::load(&[], None).expect("failed to load config");
assert_eq!(
config.listen.http,
["[::1]:9000".parse().expect("infallible")]
);
}
#[test]
fn env_list_many_elements() {
#[expect(unsafe_code, reason = "testing")]
// SAFETY: testing
unsafe {
env::set_var("AUTHENTIK_LISTEN__HTTP", "[::1]:9000,[::1]:9001");
}
let (config, _) = super::Config::load(&[], None).expect("failed to load config");
assert_eq!(
config.listen.http,
[
"[::1]:9000".parse().expect("infallible"),
"[::1]:9001".parse().expect("infallible")
]
);
}
}