Skip to content
This repository was archived by the owner on Feb 9, 2026. It is now read-only.

feat: expose a few httplib2 properties and a method - #9

Merged
busunkim96 merged 7 commits into
googleapis:masterfrom
arithmetic1728:master
Jul 7, 2020
Merged

busunkim96 merged 7 commits into
googleapis:masterfrom
arithmetic1728:master

Conversation

@arithmetic1728

@arithmetic1728 arithmetic1728 commented Jul 1, 2020 •

Copy link
Copy Markdown
Contributor

expose a few httplib2 properties and a method to help gcloud migration from httplib2 to google-auth-httplib2

Linter seems to have some problem, so I have to disable 2 rules:
(1) useless-object-inheritance: linter doesn't like class to inherits object in class AuthorizedHttp(object)
(2) wrong-import-order: linter complains wrong import order, but it contradicts itself, for instance,

tests/test_google_auth_httplib2.py:18:0: C0411: third party import 
"from six.moves import http_client" should be placed before "import mock" (wrong-import-order)

and after changing it:

tests/test_google_auth_httplib2.py:18:1: I100 Import statements are in the wrong order.
 'import mock' should be before 'from six.moves import http_client'

However, original order doesn't seem to be wrong.

@arithmetic1728
arithmetic1728 marked this pull request as ready for review July 1, 2020 21:46

@yaoliugg yaoliugg left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM

@busunkim96
busunkim96 merged commit e3aa44e into googleapis:master Jul 7, 2020
craigcitro added a commit to craigcitro/google-auth-library-python-httplib2 that referenced this pull request Jul 21, 2020
Similar to googleapis#9, this exposes the `redirect_codes` attribute of the underlying
httplib2.Http instance on AuthorizedHttp, letting users modify the set of HTTP
status codes interpreted as redirects (as in
googleapis/google-api-python-client#803).
busunkim96 pushed a commit that referenced this pull request Jul 25, 2020
Similar to #9, this exposes the `redirect_codes` attribute of the underlying
httplib2.Http instance on AuthorizedHttp, letting users modify the set of HTTP
status codes interpreted as redirects (as in
googleapis/google-api-python-client#803).
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants