Skip to content
This repository has been archived by the owner on Dec 23, 2024. It is now read-only.

feat: add env var support for logs and make config optional #36

Merged
merged 3 commits into from
Oct 24, 2024

Conversation

s0up4200
Copy link
Owner

@s0up4200 s0up4200 commented Oct 23, 2024

This PR enhances the application's configuration flexibility by:

  • Adding environment variable support for all logging settings
  • Making config.toml optional when required env vars are present
  • Adding type documentation for all env vars in docker-compose.yml

Required env vars to run without config:

REDACTEDHOOK__API_TOKEN=
REDACTEDHOOK__RED_APIKEY=
REDACTEDHOOK__OPS_APIKEY=

Testing:

  • Test starting without config.toml using env vars
  • Verify env vars override config file values

@s0up4200 s0up4200 marked this pull request as ready for review October 24, 2024 20:22
@s0up4200 s0up4200 merged commit bc909be into develop Oct 24, 2024
8 checks passed
@s0up4200 s0up4200 deleted the feat(docker)-add-more-env-var-support branch October 24, 2024 22:07
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant