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

refactor authenticationmethod settings, allow user to set authenticat… #245

Merged
merged 2 commits into from Oct 13, 2019
Merged

refactor authenticationmethod settings, allow user to set authenticat… #245

merged 2 commits into from Oct 13, 2019

Conversation

szEvEz
Copy link
Contributor

@szEvEz szEvEz commented Oct 2, 2019

Opening again, as I failed a bit on #219 🕺

…ionmethods by hhimself

Signed-off-by: szEvEz <szivos.john@gmail.com>
@rndmh3ro
Copy link
Member

rndmh3ro commented Oct 4, 2019

The tests are failing on Centos6 machines. It seems AuthenticationMethods was only introduced in OpenSSH 6.2:

  • sshd(8): Added support for multiple required authentication in SSH
    protocol 2 via an AuthenticationMethods option. This option lists
    one or more comma-separated lists of authentication method names.
    Successful completion of all the methods in any list is required for
    authentication to complete. This allows, for example, requiring a
    user having to authenticate via public key or GSSAPI before they
    are offered password authentication.

See: https://www.openssh.com/releasenotes.html

Can you add a sshd_version.stdout check to the option, please?

Signed-off-by: szEvEz <szivos.john@gmail.com>
@rndmh3ro rndmh3ro merged commit a5a1886 into dev-sec:master Oct 13, 2019
@rndmh3ro
Copy link
Member

Thanks John!

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.

None yet

2 participants