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

shellout_spec: make "current user" independent of the environment #203

Merged
merged 1 commit into from
Mar 8, 2020

Conversation

terceiro
Copy link
Contributor

@terceiro terceiro commented Mar 8, 2020

There is no guarantee that ENV["USER"] is always present, and in some
environments, it's not. Get the current user from Etc.getpwuid, which
works even in the absence of any environment variables.

@terceiro terceiro requested review from a team as code owners March 8, 2020 14:20
@lamont-granquist
Copy link
Contributor

@terceiro this commmit needs a DCO signoff for us to accept it.

There is no guarantee that ENV["USER"] is always present, and in some
environments, it's not. Get the current user from Etc.getpwuid, which
works even in the absence of any environment variables.

Signed-off-by: Antonio Terceiro <asa@terceiro.xyz>
@terceiro terceiro force-pushed the fix-test-env-user branch from aa5eade to 1d47f8f Compare March 8, 2020 22:15
@terceiro
Copy link
Contributor Author

terceiro commented Mar 8, 2020 via email

@lamont-granquist
Copy link
Contributor

thanks, merging

@lamont-granquist lamont-granquist merged commit 15b8e2c into chef:master Mar 8, 2020
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.

2 participants