-
Notifications
You must be signed in to change notification settings - Fork 188
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
Support connect_timeout keyword arg in TCPSocket #3424
Conversation
Thank you for your pull request and welcome to our community! To contribute, please sign the Oracle Contributor Agreement (OCA). To sign the OCA, please create an Oracle account and sign the OCA in Oracle's Contributor Agreement Application. When signing the OCA, please provide your GitHub username. After signing the OCA and getting an OCA approval from Oracle, this PR will be automatically updated. If you are an Oracle employee, please make sure that you are a member of the main Oracle GitHub organization, and your membership in this organization is public. |
(I'm asking to add both to the existing Shopify OCA) |
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.
It looks great, thank you.
Just one comment.
Thank you for signing the OCA. |
8b4948a
to
d99b810
Compare
d99b810
to
40b4479
Compare
Co-authored-by: Manef Zahra <manef.zahra@shopify.com> Co-authored-by: Patrick Lin <patrick.lin@shopify.com> Co-authored-by: Kevin Menard <kevin.menard@shopify.com> Co-authored-by: Randy Stauner <randy.stauner@shopify.com> closes oracle#3421
40b4479
to
e1d131d
Compare
I am not sure why, but tests/specs related to
Also one of the MRI tests. |
Accept the
connect_timeout
keyword argument to TCPSocket.newand wrap the connect call with
Timeout.timeout
when provided.closes #3421