Skip to content
This repository has been archived by the owner on Nov 8, 2021. It is now read-only.

Enhancement/centos6 support #132

Closed
wants to merge 2 commits into from
Closed

Enhancement/centos6 support #132

wants to merge 2 commits into from

Conversation

artburkart
Copy link

I'm not sure whether there'd be any interest in adding CentOS 6 support, but this commit introduces an alternative method of enabling AuthorizedKeysCommand without disabling SELinux entirely that acts as a fallback when nis_enabled is not a valid variable to set.

Additionally, since you can't upgrade openssh-server to be > 6.1 on CentOS 6, these changes use AuthorizedKeysCommandRunAs rather than AuthorizedKeysCommandUser. The way I determined whether AuthorizedKeysCommandUser was available by greping the sshd binary for the string. For some reason, I can't do the same for AuthorizedKeysCommandRunAs. I don't know where it's hiding.

Let me know what you think.

Arthur Burkart added 2 commits June 22, 2018 09:58
The default behavior was to drop the value into the codebase
regardless of whether it was supported
@michaelwittig
Copy link
Contributor

Hi @artburkart

Does this PR also includes #31 ?

I have no idea about SELinux, so let's see if someone watches this and can review it...

@artburkart
Copy link
Author

I also just noticed #97 (comment), which would mean I need to update the spec before this could go live.

@simplisafe simplisafe closed this by deleting the head repository Jun 29, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants