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

Don't use OpenSSL session cache on JRuby #38

Merged
merged 1 commit into from
Jun 25, 2024

Conversation

headius
Copy link
Contributor

@headius headius commented Jun 25, 2024

JRuby's implementation of the openssl library does not support session caching, so this results in errors and prevents the library from working.

With the given patch all but one test pass on JRuby.

See jruby/jruby#6682 (comment)

JRuby's implementation of the `openssl` library does not support
session caching, so this results in errors and prevents the
library from working.

With the given patch all but one test pass on JRuby.

See jruby/jruby#6682 (comment)
@headius
Copy link
Contributor Author

headius commented Jun 25, 2024

@shugo @hsbt If we can get this merged and released soon, that would be a big help! A user reported that all released versions of net-ftp are broken on JRuby, so we are eager to resolve that!

There's still one failure in the test suite on JRuby after this change, so I'll look at that separately after release.

@hsbt hsbt merged commit 1215c30 into ruby:master Jun 25, 2024
15 checks passed
@hsbt
Copy link
Member

hsbt commented Jun 26, 2024

@headius headius deleted the no_session_cache_on_jruby branch June 26, 2024 20:47
@headius
Copy link
Contributor Author

headius commented Jun 26, 2024

@hsbt You're the best, thanks! 🙏

headius added a commit to headius/jruby that referenced this pull request Jun 26, 2024
Previous versions were missing our patch to disable OpenSSL
session caching.

See ruby/net-ftp#38
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