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

Fix permissions when env user cannot be used for ssh login #455

Merged
merged 3 commits into from
Apr 8, 2019

Conversation

dan-blanchard
Copy link
Member

This PR makes it possible for you to use your own user account as the SSH login account for a server and then sudo to a different user for creating virtualenvs. This handles the common scenario where the superuser that you need to use to write the virtualenv is not a login account.

@dan-blanchard dan-blanchard force-pushed the fix/permissions_issues_update_virtualenv branch from 9a7a6d2 to b19d3d8 Compare March 26, 2019 14:36
@emmettbutler
Copy link
Contributor

Is there anything blocking the merge of this PR?

@dan-blanchard
Copy link
Member Author

It needs to be expanded upon for backward compatibility. I was planning on adding new settings so you can specify the user that the virtualenv creation/deletion commands run as independently of the SSH user.

@emmettbutler
Copy link
Contributor

I'm happy to make the remaining necessary changes here if you can convey some more specific idea of what's left to be done, @dan-blanchard.

@dan-blanchard
Copy link
Member Author

I can add it just as quickly as I could explain it, so I'll just bang it out.

@dan-blanchard dan-blanchard force-pushed the fix/permissions_issues_update_virtualenv branch from b19d3d8 to 5a36b5e Compare April 4, 2019 20:12
@dan-blanchard dan-blanchard force-pushed the fix/permissions_issues_update_virtualenv branch 4 times, most recently from 6986994 to 5dc2f22 Compare April 8, 2019 12:27
@dan-blanchard dan-blanchard force-pushed the fix/permissions_issues_update_virtualenv branch from 5dc2f22 to e80b327 Compare April 8, 2019 14:06
@dan-blanchard dan-blanchard merged commit 2aecc4b into master Apr 8, 2019
@dan-blanchard dan-blanchard deleted the fix/permissions_issues_update_virtualenv branch April 8, 2019 17:17
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