Allow environment variable resolution in configuration file #548

essembeh · 2022-06-16T16:55:33Z

essembeh commented

2022-06-16 16:55:33 +00:00

all string fields containing an environment variable like ${FOO} will
be resolved
supported format ${FOO}, ${FOO:-bar} and ${FOO-bar} to allow default
values if variable is not present in environment
add --no-env argument for CLI to disable the feature which is enabled
by default

Resolves: #546

- all string fields containing an environment variable like ${FOO} will be resolved - supported format ${FOO}, ${FOO:-bar} and ${FOO-bar} to allow default values if variable is not present in environment - add --no-env argument for CLI to disable the feature which is enabled by default Resolves: #546

essembeh added 1 commit 2022-06-16 16:55:33 +00:00

Allow environment variable resolution in configuration file 97b5cd089d

- all string fields containing an environment variable like ${FOO} will
  be resolved
- supported format ${FOO}, ${FOO:-bar} and ${FOO-bar} to allow default
  values if variable is not present in environment
- add --no-env argument for CLI to disable the feature which is enabled
  by default

Resolves: #546

essembeh referenced this pull request

2022-06-16 17:00:43 +00:00

Allow passing database password as environment variables. #546

witten commented

2022-06-16 18:17:46 +00:00

This looks great. And tests too! Thank you so much for taking the time to do this when all you needed was the database password from the environment. Clever use of the negative look-ahead for an escaped $.. I wouldn't have thought of that. Heads up that I may rename the --no-env flag for consistency with other borgmatic flags. And I can write up some documentation when I get a chance.

This looks great. And tests too! Thank you so much for taking the time to do this when all you needed was the database password from the environment. Clever use of the negative look-ahead for an escaped `$`.. I wouldn't have thought of that. Heads up that I may rename the `--no-env` flag for consistency with other borgmatic flags. And I can write up some documentation when I get a chance.

witten merged commit ea45f6c4c8 into master

2022-06-16 18:18:15 +00:00

essembeh referenced this issue from a commit

2022-06-17 07:52:29 +00:00

Environment variable resolution in configuration file (#546).

Sign in to join this conversation.

2 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: borgmatic-collective/borgmatic#548