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

Use SHA256 instead of SHA1 where needed in tests. #554

Merged
merged 1 commit into from Nov 23, 2022

Conversation

jackorp
Copy link
Contributor

@jackorp jackorp commented Oct 18, 2022

Systems such as RHEL 9 are moving away from SHA1
disabling it completely in default configuration.

This is a follow-up PR after #507. This PR has the goal of fixing the rest of the test failures [0] present on systems that disable SHA1, like RHEL 9 and Centos 9 stream, by default via a system-wide crypto policy.

Tests pass locally with these changes.

[0] https://gist.github.com/jackorp/bb6e3ff2cfd339f2f65afcdc5c9e4070

This supersedes #511

Systems such as RHEL 9 are moving away from SHA1
disabling it completely in default configuration.
@jackorp
Copy link
Contributor Author

jackorp commented Oct 18, 2022

I decided to redo the change completely instead of trying to fix I had done in #511 .

@rhenium rhenium merged commit 17fb6f1 into ruby:master Nov 23, 2022
@rhenium
Copy link
Member

rhenium commented Nov 23, 2022

Thank you!

hsbt pushed a commit to ruby/spec that referenced this pull request Jan 5, 2023
Systems such as CentOS 9 / RHEL 9 are moving away from SHA1 disabling it
by default via a system-wide crypto policy. This replaces SHA1 with
SHA256 in similar way as [[1]].

[1]: ruby/openssl#554
seven1m pushed a commit to seven1m/ruby_spec that referenced this pull request Sep 2, 2023
Systems such as CentOS 9 / RHEL 9 are moving away from SHA1 disabling it
by default via a system-wide crypto policy. This replaces SHA1 with
SHA256 in similar way as [[1]].

[1]: ruby/openssl#554
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants