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

TEST: Current value of ssh_hash_known_hosts causes error in the defalt configuration in FIPS mode #5996

Closed
wants to merge 1 commit into from

Conversation

dparmar18
Copy link

Explanation

  • In SSSD the default value for ssh_hash_known_hosts is set to true,
    It should be changed to false for consistency with the OpenSSH
    setting that does not hash hostnames by default

Verifies
Issue: #5848
Bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=2014249

@alexey-tikhonov
Copy link
Member

alexey-tikhonov commented Feb 15, 2022

@sgoveas, could you please take a look at the updated version?

This is instead of #5925

src/tests/multihost/ipa/test_misc.py Outdated Show resolved Hide resolved
src/tests/multihost/ipa/test_misc.py Outdated Show resolved Hide resolved
src/tests/multihost/ipa/test_misc.py Outdated Show resolved Hide resolved
@sgoveas
Copy link
Contributor

sgoveas commented Feb 19, 2022

Fix the pep8 errors

…lt configuration in FIPS mode

Explanation
- In SSSD the default value for ssh_hash_known_hosts is set to true,
  It should be changed to false for consistency with the OpenSSH
  setting that does not hashes host names by default

Verifies
  Issue: SSSD#5848
  Bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=2014249
@dparmar18
Copy link
Author

Fix the pep8 errors

Done

@pbrezina pbrezina added the Ready to push Ready to push label Feb 22, 2022
@pbrezina
Copy link
Member

Pushed PR: #5996

  • master
    • d082681 - TEST: Current value of ssh_hash_known_hosts causes error in the default configuration in FIPS mode

@pbrezina pbrezina added Pushed and removed Accepted Ready to push Ready to push labels Feb 22, 2022
@pbrezina pbrezina closed this Feb 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants