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 "A temporary workaround to download OpenSSL archive files." #781

Merged

Conversation

junaruga
Copy link
Member

@junaruga junaruga commented Jul 24, 2024

I was told that OpenSSL project resolved the website issue at openssl/openssl#24984 (comment).

This reverts commit f2c7729.

I checked this reverting commit worked on the CI in my forked repository.
https://github.com/junaruga/ruby-openssl/actions/runs/10080561249

I was told that OpenSSL project resolved the website issue at
<openssl/openssl#24984 (comment)>.

This reverts commit f2c7729.
@rhenium
Copy link
Member

rhenium commented Jul 24, 2024

Good to hear!

Regardless, I wonder if we should switch to using GitHub for newer OpenSSL releases if it now is the primary location.

@rhenium rhenium merged commit 3b1838f into ruby:master Jul 24, 2024
53 of 54 checks passed
@junaruga junaruga deleted the wip/revert-openssl-source-file-workaround branch July 24, 2024 16:58
@junaruga
Copy link
Member Author

Regardless, I wonder if we should switch to using GitHub for newer OpenSSL releases if it now is the primary location.

Yes, as the GitHub releases are the primary locations to download, using GitHub releases makes sense to me.

As I said at #778 (comment), the links to the openssl-1.0.2u.tar.gz and openssl-1.1.0l.tar.gz are missing on https://github.com/openssl/openssl/releases . And it seems the link format is different between OpenSSL 3.x and OpenSSL 1.1.1w. However, we may be able to apply the links to the https://github.com/openssl/openssl/releases conditionally only for the OpenSSL 3.x.

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.

2 participants