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

Update SSH requirements #4225

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Update SSH requirements #4225

wants to merge 2 commits into from

Conversation

rahendatri
Copy link

add other sshd_config params

For Rancher (product) docs only

When contributing to docs, please update the versioned docs. For example, the docs in the v2.6 folder of the rancher folder.

Doc versions older than the latest minor version should only be updated to fix inaccuracies or make minor updates as necessary. The majority of new content should be added to the folder for the latest minor version.

add other sshd_config params
@rahendatri
Copy link
Author

closes rancher/rke#2907

@jtravee jtravee requested a review from kinarashah July 27, 2022 14:19
content/rke/latest/en/os/_index.md Outdated Show resolved Hide resolved
@jtravee jtravee added the RKE Issues related to RKE docs label Jul 28, 2022
@rahendatri
Copy link
Author

rahendatri commented Sep 20, 2022

@kinarashah @jtravee Hello, it seems like this PR is still left unmerged.
The requested change has been taken into account. Would you be able to process this ?
Thanks in advance.

@snasovich
Copy link
Contributor

snasovich commented Jan 21, 2023

@jtravee @kinarashah , please review this when you have a chance.

Additional context: Asked to check if this helps couple other users that are running into this issue: rancher/rke#2941 (comment)
@kinarashah , the scope for your review is mostly confirming if we're OK with recommending these settings (suggested setting values are defauls per https://man.openbsd.org/sshd_config, but AllowTcpForwarding yes we're already recommending to set is the default as well, so there is probably a reason why we need to set it).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
RKE Issues related to RKE docs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants