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

Docker secrets? #11

Closed
jathek opened this issue Dec 2, 2020 · 4 comments
Closed

Docker secrets? #11

jathek opened this issue Dec 2, 2020 · 4 comments

Comments

@jathek
Copy link

jathek commented Dec 2, 2020

Any plans to implement docker secrets for the username and password? Ideally I could use secrets for the u/p for both PIA and the proxies.

thrnz added a commit that referenced this issue Dec 2, 2020
…from existing files if needed (eg Docker secrets)

See issue #11
@thrnz
Copy link
Owner

thrnz commented Dec 2, 2020

I'm not familiar with using Docker secrets, but I've done what the docs suggest and have added USER_FILE and PASS_FILE vars so the user/pass can be read in from a file if needed.

I've briefly tested using USER_FILE=/run/secrets/pia-username and PASS_FILE=/run/secrets/pia-password along with corresponding secrets and it seems to be working.

Let me know if theres any wonkiness, or if adding any more vars would be useful.

@jathek
Copy link
Author

jathek commented Dec 2, 2020

Just tested it, worked great.

The only other suggestion I would have for vars is unrelated, but your container allows for local network exceptions in a space-separated environment variable. It's such a small thing, but most of the other containers I've seen do that as a comma-separated variable? If yours used commas too then I could put my local exceptions into my .env file and maintain it there instead of in a couple different places in my compose file.

@thrnz
Copy link
Owner

thrnz commented Dec 2, 2020

I could probably just make it replace the commas with spaces so it can accept either. I don't think that would break anything.

@jathek
Copy link
Author

jathek commented Dec 2, 2020

I could probably just make it replace the commas with spaces so it can accept either. I don't think that would break anything.

Cheers, that would work. I was trying to figure out a command to run to do it myself but I'm still learning shell and docker and stuff.

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

No branches or pull requests

2 participants