Skip to content

/ saltfs

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

Deploy Mozilla-recommended sshd config #254

Closed
wants to merge 1 commit into from
Closed

Deploy Mozilla-recommended sshd config #254

wants to merge 1 commit into from

Conversation

@edunham
Copy link
Contributor

edunham commented Mar 15, 2016

See https://wiki.mozilla.org/Security/Guidelines/OpenSSH

r? @aneeshusa

Please do not merge until #253 is landed and verified working by everyone. These changes will disallow root login.

This change is Reviewable
@edunham
Deploy Mozilla-recommended SSHD config
Loading status checks…
057272a 
See https://wiki.mozilla.org/Security/Guidelines/OpenSSH
@edunham
Copy link
Contributor Author

edunham commented Mar 15, 2016

Failure is due to root group being unavailable on Mac. Does Salt have a platform-agnostic way of specifying the correct perms or inheriting them from the directory?
@aneeshusa
Copy link
Member

aneeshusa commented Mar 15, 2016

This has actually been on my mind for a while (and is something I've been investigating), and a robust fix is going to be quite a bit more involved. I'll cycle back to this after #253 is landed.
@aneeshusa aneeshusa mentioned this pull request Mar 15, 2016
Closed
@aneeshusa aneeshusa mentioned this pull request Jun 27, 2016
Open
@bors-servo
Copy link
Contributor

bors-servo commented Jan 17, 2017

☔ The latest upstream changes (presumably #577) made this pull request unmergeable. Please resolve the merge conflicts.
@edunham
Copy link
Contributor Author

edunham commented Mar 27, 2017

Fixed the blocker of my Salt ignorance here by cribbing off how the admin state handles the hosts file.

Cherry-picked and closing in favor of #628
@edunham edunham closed this Mar 27, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
DO NOT MERGE YET S-needs-rebase
Projects
None yet
Milestone
No milestone
Linked issues

Successfully merging this pull request may close these issues.

None yet

4 participants
@edunham @aneeshusa @bors-servo @highfive
You can’t perform that action at this time.