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

fix: Workaround for CentOS 10 issues in openssh #167

Merged
merged 3 commits into from
Jan 8, 2025
Merged
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Workaround wrong parsing of compression
This was reported upstream and fixed:

https://bugzilla.mindrot.org/show_bug.cgi?id=3759

But the compression is something we do not want to use anyway much so
implementing this workaround similar to the one we did in the server role:

willshersystems/ansible-sshd#304

Signed-off-by: Jakub Jelen <jjelen@redhat.com>
Jakuje committed Jan 8, 2025
commit 9a2287c7129e6e890ceddfd1b80c8510c5996f27
10 changes: 5 additions & 5 deletions tests/tests_match.yml
Original file line number Diff line number Diff line change
@@ -19,7 +19,7 @@
ssh:
Match:
- Condition: user testuser
Compression: true
LogLevel: DEBUG2
- Condition: host example
Hostname: example.com
Host:
@@ -39,7 +39,7 @@
assert:
that:
- "'Match user testuser' in config.content | b64decode"
- "'Compression yes' in config.content | b64decode"
- "'LogLevel DEBUG2' in config.content | b64decode"
- "'Match host example' in config.content | b64decode"
- "'Hostname example.com' in config.content | b64decode"
- "'Host local' in config.content | b64decode"
@@ -70,9 +70,9 @@
- name: Verify the options are correctly evaluated
assert:
that:
- "'compression yes' in test1.stdout"
- "'compression yes' not in test2.stdout"
- "'compression yes' not in test3.stdout"
- "'loglevel DEBUG2' in test1.stdout"
- "'loglevel DEBUG2' not in test2.stdout"
- "'loglevel DEBUG2' not in test3.stdout"
- "'hostname example.com' in test1.stdout"
- "'hostname example.com' not in test2.stdout"
- "'hostname example.com' not in test3.stdout"