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

Add HOME env var to ipa-server-configure.service #184

Merged
merged 1 commit into from Jan 12, 2018

Conversation

stlaz
Copy link
Contributor

@stlaz stlaz commented Jan 11, 2018

OpenSSL requires at least one of HOME or RANDFILE environment
variables to be set for it to work.

Without this fix, the external CA setup crashes because OpenSSL
is unable to write its random state anywhere when it tries to
generate a CSR for the external CA to sign.

@adelton
Copy link
Collaborator

adelton commented Jan 11, 2018

Reading man systemd.exec(5) I don't see expansion of tilda documented.

@adelton
Copy link
Collaborator

adelton commented Jan 11, 2018

OTOH, there is

       $USER, $LOGNAME, $HOME, $SHELL
           User name (twice), home directory, and the login shell. The
           variables are set for the units that have User= set, which
           includes user systemd instances. See passwd(5).

so maybe User=root would do the trick?

@stlaz
Copy link
Contributor Author

stlaz commented Jan 11, 2018

That also works and is probably cleaner. Rebased and updated the commit message.

@adelton
Copy link
Collaborator

adelton commented Jan 11, 2018

Based on https://bugzilla.redhat.com/show_bug.cgi?id=1524833#c28, maybe RANDFILE=/dev/null would actually be the best approach. Data that we don't store cannot cause problems during upgrades, for example. ;-)

OpenSSL requires at least one of HOME or RANDFILE environment
variables to be set for it to work.

Without this fix, the external CA setup crashes because OpenSSL
is unable to write its random state anywhere when it tries to
generate a CSR for the external CA to sign.
@stlaz
Copy link
Contributor Author

stlaz commented Jan 12, 2018

Updated and deployed testing.

@stlaz
Copy link
Contributor Author

stlaz commented Jan 12, 2018

Tests passed, merging.

@stlaz stlaz merged commit bd3a33a into freeipa:master Jan 12, 2018
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