Skip to content

Support env variables inside config variables#232

Merged
lowercasename merged 9 commits into
lowercasename:mainfrom
halkeye:support-env-in-config
Mar 4, 2026
Merged

Support env variables inside config variables#232
lowercasename merged 9 commits into
lowercasename:mainfrom
halkeye:support-env-in-config

Conversation

@halkeye

@halkeye halkeye commented Nov 2, 2025

Copy link
Copy Markdown
Contributor

so
site_name=$GATHIO_USER}'s event site
will get pouplated by process.env.GATHIO_USER

For the docker container. I would like to inject credentials from outside. I like having my config file public in my devops repo, then any secrets are hidden.

There's other ways for me to do it if you don't want it.

@halkeye

halkeye commented Nov 2, 2025

Copy link
Copy Markdown
Contributor Author

oh i see #219 is another way to do it.

@halkeye

halkeye commented Nov 2, 2025

Copy link
Copy Markdown
Contributor Author

Okay, based on #219, i moved it to after toml parsing. I think its way more complicated this way
Also just restricted it to GATHIO_ variables, so it can't leak any system configs

site_name=$USER}'s event site
will get pouplated by process.env.GATHIO_USER
@halkeye halkeye force-pushed the support-env-in-config branch from 671547a to 80749b1 Compare November 14, 2025 06:01
@halkeye

halkeye commented Nov 14, 2025

Copy link
Copy Markdown
Contributor Author

@lowercasename rebased after prettier

@halkeye

halkeye commented Feb 25, 2026

Copy link
Copy Markdown
Contributor Author

:nudge: @lowercasename

@lowercasename

Copy link
Copy Markdown
Owner

Apolgoies! This is next!

@halkeye

halkeye commented Feb 26, 2026

Copy link
Copy Markdown
Contributor Author

[warn] src/activitypub.ts
[warn] src/lib/activitypub.ts
[warn] Code style issues found in 2 files. Run Prettier to fix.

do you want me to do that on this pr? or a seperate pr?

@halkeye

halkeye commented Mar 4, 2026

Copy link
Copy Markdown
Contributor Author

excelllanttttttttttttt
we so close

@lowercasename

Copy link
Copy Markdown
Owner

Apologies for taking so long to get to this - I felt there was something not quite there with the config object parsing and it apparently required a quiet train ride for me to get to the bottom of it. This should now work nicely.

Comment thread src/lib/config.ts Outdated
Comment thread src/lib/config.ts Outdated
lowercasename and others added 3 commits March 4, 2026 21:42
Co-authored-by: Gavin Mogan <github@gavinmogan.com>
Co-authored-by: Gavin Mogan <github@gavinmogan.com>
@lowercasename lowercasename merged commit 61acc0d into lowercasename:main Mar 4, 2026
1 check passed
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.

2 participants