[GH-ISSUE #963] [Feature request] Allow yml to be absent when ENV is set #10468

Closed
opened 2026-05-06 13:48:18 -05:00 by GiteaMirror · 6 comments
Owner

Originally created by @jackrosenberg on GitHub (Jun 24, 2025).
Original GitHub issue: https://github.com/fosrl/pangolin/issues/963

In config.yml, certain fields such as USERS_SERVERADMIN_PASSWORD need to be hard coded. It would be nice if these fields could be left blank in the case that the corresponding env var is found on startup. Currently pangolin throws: Error: Invalid configuration file: Validation error, which is the expected behavior, but something like lazy loading would be an elegant fix for this.

Originally created by @jackrosenberg on GitHub (Jun 24, 2025). Original GitHub issue: https://github.com/fosrl/pangolin/issues/963 In `config.yml`, certain fields such as `USERS_SERVERADMIN_PASSWORD` need to be hard coded. It would be nice if these fields could be left blank in the case that the corresponding `env` var is found on startup. Currently pangolin throws: `Error: Invalid configuration file: Validation error`, which is the expected behavior, but something like lazy loading would be an elegant fix for this.
GiteaMirror added the enhancementnon-critical bughelp wanted labels 2026-05-06 13:48:19 -05:00
Author
Owner

@oschwartz10612 commented on GitHub (Jun 25, 2025):

Good point this should be a simple fix we could add soon!

<!-- gh-comment-id:3005116375 --> @oschwartz10612 commented on GitHub (Jun 25, 2025): Good point this should be a simple fix we could add soon!
Author
Owner

@jackrosenberg commented on GitHub (Jun 25, 2025):

Thanks for the fast response, I can PR this myself sometime in the coming weeks :)

<!-- gh-comment-id:3005205906 --> @jackrosenberg commented on GitHub (Jun 25, 2025): Thanks for the fast response, I can PR this myself sometime in the coming weeks :)
Author
Owner

@oschwartz10612 commented on GitHub (Jun 26, 2025):

That would be great!

<!-- gh-comment-id:3008828315 --> @oschwartz10612 commented on GitHub (Jun 26, 2025): That would be great!
Author
Owner

@jackrosenberg commented on GitHub (Jun 30, 2025):

Think this is closed by 1.6.0, right? @oschwartz10612

<!-- gh-comment-id:3020419913 --> @jackrosenberg commented on GitHub (Jun 30, 2025): Think this is closed by 1.6.0, right? @oschwartz10612
Author
Owner

@miloschwartz commented on GitHub (Jul 1, 2025):

@jackrosenberg Yep! We removed the need for these variables in 1.6

<!-- gh-comment-id:3021686319 --> @miloschwartz commented on GitHub (Jul 1, 2025): @jackrosenberg Yep! We removed the need for these variables in 1.6
Author
Owner

@jackrosenberg commented on GitHub (Jul 1, 2025):

Nice, good one!

<!-- gh-comment-id:3022564888 --> @jackrosenberg commented on GitHub (Jul 1, 2025): Nice, good one!
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/pangolin#10468