Skip to content

fix(config): support list options from environment variables - #1399

Merged
lovasoa merged 3 commits into
mainfrom
fix/config-list-env-1383
Aug 23, 2026
Merged

fix(config): support list options from environment variables#1399
lovasoa merged 3 commits into
mainfrom
fix/config-list-env-1383

Conversation

@lovasoa

@lovasoa lovasoa commented Aug 23, 2026

Copy link
Copy Markdown
Collaborator

Fixes #1383

Summary

Register every list-valued AppConfig option with the config crate environment parser. Environment variables now accept space-separated lists for sqlite_extensions, oidc_protected_paths, oidc_public_paths, and oidc_additional_trusted_audiences. The configuration reference documents the environment syntax.

Stripped verification logs

$ SQLPAGE_OIDC_PROTECTED_PATHS="/user /admin" SQLPAGE_OIDC_PUBLIC_PATHS="/health /login" SQLPAGE_OIDC_ADDITIONAL_TRUSTED_AUDIENCES="audience_one audience_two" ./target/debug/sqlpage
Loaded configuration: ... oidc_protected_paths: ["/user", "/admin"], oidc_public_paths: ["/health", "/login"], oidc_additional_trusted_audiences: Some(["audience_one", "audience_two"]) ...
SQLPage v0.45.0 started successfully!

Full cargo test passed: 175 unit tests and 73 integration tests. cargo clippy --all-targets --all-features -- -D warnings and cargo fmt --all -- --check also passed.

@lovasoa
lovasoa merged commit 0f7eba5 into main Aug 23, 2026
8 checks passed
@lovasoa
lovasoa deleted the fix/config-list-env-1383 branch August 23, 2026 15:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

List-typed config options (oidc_protected_paths, oidc_additional_trusted_audiences) cannot be set via environment variables

1 participant