Skip to content
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.

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

_FILE variables for Docker swarm secrets #1607

Merged
merged 3 commits into from Mar 8, 2021
Merged

_FILE variables for Docker swarm secrets #1607

merged 3 commits into from Mar 8, 2021

Conversation

lub
Copy link
Member

@lub lub commented Aug 29, 2020

What type of PR?

enhancement

What does this PR do?

This PR enables usage of DB_PW_FILE and SECRET_KEY_FILE instead of DB_PW and SECRET_KEY to load these values from files instead of supplying them directly. That way it's possible to use Docker secrets.

Related issue(s)

Prerequistes

Before we can consider review and merge, please make sure the following list is done and checked.
If an entry in not applicable, you can check it or remove it from the list.

  • In case of feature or enhancement: documentation updated accordingly
  • Unless it's docs or a minor change: add changelog entry file.

this enables usage of e.g. docker swarm secrets instead of exposing the
passwords directly via environment variables

just use DB_PW_FILE and SECRET_KEY_FILE instead of DB_PW and SECRET_KEY
@mergify
Copy link
Contributor

mergify bot commented Aug 29, 2020

Thanks for submitting this pull request.
Bors-ng will now build test images. When it succeeds, we will continue to review and test your PR.

bors try

Note: if this build fails, read this.

bors bot added a commit that referenced this pull request Aug 29, 2020
@lub lub changed the title Swarm secrets _FILE variables for Docker swarm secrets Aug 29, 2020
@bors
Copy link
Contributor

bors bot commented Aug 29, 2020

try

Build succeeded:

Copy link
Member

@Diman0 Diman0 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me

@mergify
Copy link
Contributor

mergify bot commented Mar 8, 2021

bors r+

@bors
Copy link
Contributor

bors bot commented Mar 8, 2021

Build succeeded:

@bors bors bot merged commit cca4b50 into Mailu:master Mar 8, 2021
nextgens pushed a commit to nextgens/Mailu that referenced this pull request Jun 4, 2021
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.

None yet

2 participants