Skip to content
This repository has been archived by the owner on Dec 26, 2020. It is now read-only.

Parameterise Banner and DebianBanner as defaults #77

Merged
merged 2 commits into from
Oct 24, 2016

Conversation

tsenart
Copy link
Contributor

@tsenart tsenart commented Oct 12, 2016

This commit parameterises Banner and DebianBanner in the sshd config
with the defaults ssh_banner and ssh_os_banner respectively, though the
latter is only supported on Debian family distributions.

Debian based distributions will have DebianBanner no set by default
from now on.

@rndmh3ro
Copy link
Member

Thanks for that! Looks good.
One nitpick: Could you rename the variable ssh_os_banner to ssh_show_debian_banner or something? Makes it more clear what this does from the name of the variable.

@tsenart
Copy link
Contributor Author

tsenart commented Oct 17, 2016

@rndmh3ro: OK, I'll rename it.

This commit parameterises `Banner` and `DebianBanner` in the sshd config
with the defaults `ssh_banner` and `ssh_print_debian_banner` respectively,
though the latter is only supported on Debian family distributions.

Debian based distributions will have `DebianBanner no` set by default
from now on.
@rndmh3ro rndmh3ro merged commit 1f63b35 into dev-sec:master Oct 24, 2016
@rndmh3ro
Copy link
Member

Thanks!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants