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

Ruby - stop using an obsolete method of Net::HTTP #8408

Merged
merged 3 commits into from
Jun 10, 2020

Conversation

masakazutakewaka
Copy link
Contributor

@masakazutakewaka masakazutakewaka commented Jun 9, 2020

Description

Net::HTTP::Proxy is an obsolete method, so replaced it with Net::HTTP.new as the doc says -> https://docs.ruby-lang.org/en/2.7.0/Net/HTTP.html#method-c-Proxy

Motivation and Context

To keep the code up to date.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist

  • I have read the contributing document.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@@ -120,16 +120,16 @@ def latest
end
end

def net_http
def net_http_start(address, &block)
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should I write about this in the CHANGE.log as this method is exposed to public?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It might worth mentioning, but frankly, I don't think this was ever intended to be a public API.

@p0deje p0deje added the C-rb label Jun 9, 2020
Copy link
Member

@p0deje p0deje left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for the contribution, I'll merge as soon as CI is green!

  https://docs.ruby-lang.org/en/2.7.0/Net/HTTP.html#method-c-Proxy

Net::HTTP::Proxy is an obsolete method, so replaced it with
Net::HTTP.new as the doc above says
@masakazutakewaka masakazutakewaka force-pushed the rb-replace-obsolete-method branch from f1a69e0 to 2ab56e1 Compare June 10, 2020 03:28
@p0deje p0deje merged commit 03bce95 into SeleniumHQ:master Jun 10, 2020
@p0deje
Copy link
Member

p0deje commented Jun 10, 2020

Thank you!

titusfortner pushed a commit to titusfortner/selenium that referenced this pull request Aug 13, 2020
https://docs.ruby-lang.org/en/2.7.0/Net/HTTP.html#method-c-Proxy

Net::HTTP::Proxy is an obsolete method, so replaced it with
Net::HTTP.new as the doc above says
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants