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

Pass environment needed for systemd-ask-pass #630

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Firstyear
Copy link

systemd-ask-pass allows the pin to be provided by the systemd-credentials mechanism, but that requires the environment variable $CREDENTIALS_DIRECTORY to be set. Openvpn when launched by systemd does have this environment variable available, but it doesn't pass it through to systemd-ask-pass meaning that systemd-ask-pass is unable to load the pin from
systemd-credentials.

This change passes the $CREDENTIALS_DIRECTORY environment variable to the execed systemd-ask-pass invocation so that
systemd-credentials can be used for non-interactive pin entry.

Thank you for your contribution

You are welcome to open PR, but they are used for discussion only. All
patches must eventually go to the openvpn-devel mailing list for review:

Please send your patch using git-send-email. For example to send your latest commit to the list:

$ git send-email [email protected] HEAD~1

For details, see these Wiki articles:

systemd-ask-pass allows the pin to be provided by the
systemd-credentials mechanism, but that requires the environment
variable $CREDENTIALS_DIRECTORY to be set. Openvpn when launched
by systemd does have this environment variable available, but it
doesn't pass it through to systemd-ask-pass meaning that
systemd-ask-pass is unable to load the pin from
systemd-credentials.

This change passes the $CREDENTIALS_DIRECTORY environment variable
to the execed systemd-ask-pass invocation so that
systemd-credentials can be used for non-interactive pin entry.

Signed-off-by: William <[email protected]>
@Firstyear
Copy link
Author

I won't be able to submit this to a mailing list, as I don't have a git email work flow setup. Feel free to send this on my behalf.

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.

1 participant