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

Support a Requests_Proxy_HTTP() instance for the proxy setting. #223

Merged
merged 3 commits into from
Jul 12, 2016

Conversation

@rmccue rmccue added this to the 1.7 milestone Jul 12, 2016
@rmccue rmccue added the Extras label Jul 12, 2016
@rmccue
Copy link
Collaborator

rmccue commented Jul 12, 2016

@ocean90 Can you fix the unit test for this too please? RequestsTest_Proxy_HTTP::testConnectWithInstance should have been testing this, but looks like a copypasta issue.

@ocean90
Copy link
Member Author

ocean90 commented Jul 12, 2016

@rmccue On it.

@codecov-io
Copy link

codecov-io commented Jul 12, 2016

Current coverage is 92.22%

Merging #223 into master will increase coverage by 0.02%

@@             master       #223   diff @@
==========================================
  Files            21         21          
  Lines          1769       1761     -8   
  Methods         156        156          
  Messages          0          0          
  Branches          0          0          
==========================================
- Hits           1631       1624     -7   
+ Misses          138        137     -1   
  Partials          0          0          

Powered by Codecov. Last updated by 2781cdd...1c5455d

@rmccue
Copy link
Collaborator

rmccue commented Jul 12, 2016

Wonderful, thanks!

@rmccue rmccue merged commit 539cdfd into WordPress:master Jul 12, 2016
markjaquith pushed a commit to markjaquith/WordPress that referenced this pull request Jul 13, 2016
Fixes an issue where you couldn't set a `Requests_Proxy_HTTP` object as a proxy setting.

See WordPress/Requests#223.
See #37107, #33055.
Built from https://develop.svn.wordpress.org/trunk@38053


git-svn-id: http://core.svn.wordpress.org/trunk@37994 1a063a9b-81f0-0310-95a4-ce76da25c4cd
@ocean90 ocean90 deleted the proxy-instance branch July 13, 2016 16:01
svn2github pushed a commit to svn2github/wp-svn-2-git that referenced this pull request Jul 13, 2016
Fixes an issue where you couldn't set a `Requests_Proxy_HTTP` object as a proxy setting.

See WordPress/Requests#223.
See #37107, #33055.
Built from https://develop.svn.wordpress.org/trunk@38053


git-svn-id: http://core.svn.wordpress.org/trunk@37994 1a063a9b-81f0-0310-95a4-ce76da25c4cd
dustinrue pushed a commit to openfcci/wordpress that referenced this pull request Jul 14, 2016
Fixes an issue where you couldn't set a `Requests_Proxy_HTTP` object as a proxy setting.

See WordPress/Requests#223.
See #37107, #33055.
Built from https://develop.svn.wordpress.org/trunk@38053


git-svn-id: http://core.svn.wordpress.org/trunk@37994 1a063a9b-81f0-0310-95a4-ce76da25c4cd
staylor pushed a commit to staylor/develop.svn.wordpress.org that referenced this pull request Aug 18, 2016
Fixes an issue where you couldn't set a `Requests_Proxy_HTTP` object as a proxy setting.

See WordPress/Requests#223.
See #37107, #33055.

git-svn-id: https://develop.svn.wordpress.org@38053 602fd350-edb4-49c9-b593-d223f7449a82
nylen pushed a commit to nylen/wordpress-develop-svn that referenced this pull request Feb 16, 2017
Fixes an issue where you couldn't set a `Requests_Proxy_HTTP` object as a proxy setting.

See WordPress/Requests#223.
See #37107, #33055.

git-svn-id: https://develop.svn.wordpress.org/trunk@38053 602fd350-edb4-49c9-b593-d223f7449a82
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.

3 participants