Skip to content

Support for docker secrets and referenced vars #190

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 3 commits into
base: develop
Choose a base branch
from
Open

Support for docker secrets and referenced vars #190

wants to merge 3 commits into from

Conversation

greganagno
Copy link

Main additions:

  • Docker secrets are loaded as environment variables
  • Referenced env variables can be used in db strings, cache strings, etc.
  • Tests are included

@greganagno
Support for docker secrets and referenced vars
a1aeb97 
Main additions:
- Docker secrets are loaded as environment variables
- Referenced env variables can be used in db strings, cache strings, etc.
- Tests are included
@coveralls
Copy link

coveralls commented Jul 7, 2018 •
edited
Loading

Coverage Status

Coverage increased (+2.5%) to 88.261% when pulling e069585 on greganagno:feature/read_docker_secrets into c262002 on joke2k:develop.

greganagno added 2 commits July 7, 2018 20:54
@greganagno
Remove the file extension of the docker secret files
f1b0101
@greganagno
Read docker secrets directly from the disk
e069585 
Previously the docker secrets were loaded as environment variables
@joke2k joke2k added the enhancement New feature or request label Mar 15, 2020
@sergeyklay sergeyklay deleted the branch joke2k:develop September 4, 2021 12:00
@sergeyklay sergeyklay closed this Sep 4, 2021
@sergeyklay sergeyklay reopened this Sep 4, 2021
@sergeyklay
Copy link
Collaborator

PR was closed by mistake. Reopened it.

@sergeyklay sergeyklay deleted the branch joke2k:develop October 19, 2021 21:45
@sergeyklay sergeyklay closed this Oct 19, 2021
@sergeyklay sergeyklay reopened this Oct 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@greganagno @coveralls @sergeyklay @joke2k