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
Add IO#timeout attribute and use it for blocking IO operations. #5653
Conversation
7c695e8
to
8f5a57e
Compare
1145393
to
5509ca0
Compare
e1be1c4
to
28890f0
Compare
1b6a908
to
4070940
Compare
5212173
to
2361573
Compare
Consider the backward compatibility of ruby/openssl. |
2f8b0df
to
2e57446
Compare
6438434
to
32e1a4f
Compare
|
||
/** | ||
* Set the timeout associated with the specified io object. This timeout is | ||
* used as a best effort timeout to prevent operations from blocking forever. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I recommend to denote examples like "socket read/write".
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ah, it's not a description for the method.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry, I don't fully understand your feedback, but I'm happy to improve the documentation.
I have a follow up PR to improve the usage for backwards compatible extensions: #6507 |
Update to openssl gem: ruby/openssl#547 |
@@ -14,14 +14,22 @@ | |||
} | |||
end | |||
|
|||
ruby_version_is "3.0" do | |||
ruby_version_is "3.0"..."3.1" do |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should have been ..."3.2"
, fixed in ruby/spec@c3677cf
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
https://bugs.ruby-lang.org/issues/18630