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

Revert "fixed gnutls as per issue 196 thansk to @jmalpede" #234

Conversation

qwestduck
Copy link
Contributor

@qwestduck qwestduck commented Oct 24, 2023

This reverts commit 63c4c84.

Overall Review of Changes:

Reversion of a previous commit that introduced a regression.

Issue Fixes:

Fixes #233

How has this been tested?:

File content verified after running playbook.

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Congrats on opening your first pull request and thank you for taking the time to help improve Ansible-Lockdown!
Please join in the conversation happening on the Discord Server as well.

@uk-bolly
Copy link
Member

hi @qwestduck

Thank you for raising this issue, unfortunately since the ansible galaxy changes this has broken older ansible releases. We are trying to give clients a longer support around older ansible versions. This means we need to update the collections method for the pipeline.
I am raising this and pushing a new branch today into devel with these fixes.

many thanks

uk-bolly

This reverts commit 63c4c84.

Signed-off-by: William Panlener <wpanlener@gmail.com>
@qwestduck qwestduck force-pushed the 233-gnutls-crypto-policy-should-not-omit-vers-all branch from e37b47e to 15f8d9d Compare October 24, 2023 18:58
Copy link
Member

@uk-bolly uk-bolly left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great PR

@uk-bolly uk-bolly merged commit 96bf4af into ansible-lockdown:devel Nov 1, 2023
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

RHEL-08-010295 omits '+VERS-ALL' from gnutls crypto policy
2 participants