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 support for sshd_config include files #390

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

nvergottini
Copy link

Add include_dir parameter for specifying an include directory at the top of sshd_config.

Add ssh::server::config_file resource type for creating config files within the include directory. Provides include parameter for including externally managed config files. This is primarily intended for including crypto policies in RedHat 9 family.

Add data for RedHat 9 family to add include directory and config file to load crypto policies for OpenSSH server by default.

Add include_dir parameter for specifying an include directory at the top
of sshd_config.

Add ssh::server::config_file resource type for creating config files
within the include directory. Provides include parameter for including
externally managed config files. This is primarily intended for
including crypto policies in RedHat 9 family.

Add data for RedHat 9 family to add include directory and config file to
load crypto policies for OpenSSH server by default.
@nvergottini
Copy link
Author

nvergottini commented Jun 20, 2024

I developed this change after I discovered that using crypto policies on Oracle Linux 9 to manage sshd crypto settings was not working like it did with Oracle Linux 8. Without the "Include /etc/ssh/sshd_config.d/*.conf" at the top of sshd_config, the "Include /etc/crypto-policies/back-ends/opensshserver.config" at the top of /etc/ssh/sshd_config.d/50-redhat.conf was not being loaded, so crypto policies were not being applied. EL8 uses a completely different mechanism to manage sshd crypto settings that was not affected by the use of this module.

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.

1 participant