Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(96)

Issue 7033052: Adding a .revoke() to Credentials. (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
12 years, 6 months ago by dhermes
Modified:
12 years, 5 months ago
Reviewers:
jcgregorio_google
CC:
google-api-python-client_googlegroups.com
Visibility:
Public.

Patch Set 1 #

Patch Set 2 : Removing revoke_uri as required in clientsecrets and making Google constants strings (were tuples). #

Patch Set 3 : Adding necessary revoke method and helpers, in addition to a small refactor. #

Patch Set 4 : Change finished for review, added remaining tests. #

Patch Set 5 : Disabling revoke for assertion credentials and subclasses. #

Total comments: 4

Patch Set 6 : Removed unneccessary revoke override in AppAssertionCredentials. #

Total comments: 8

Patch Set 7 : Updating tests per review comments. #

Patch Set 8 : Adding revoke support for assertion credentials after verifying it works. #

Total comments: 4

Patch Set 9 : Removed token from logging call. #

Patch Set 10 : Pulled and updated from tip before committing. #

Patch Set 11 : Pulled and updated from tip before committing. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+387 lines, -156 lines) Patch
M oauth2client/__init__.py View 1 1 chunk +4 lines, -0 lines 0 comments Download
M oauth2client/appengine.py View 1 2 3 4 5 7 chunks +24 lines, -12 lines 0 comments Download
M oauth2client/client.py View 1 2 3 4 5 6 7 8 36 chunks +180 lines, -58 lines 0 comments Download
M oauth2client/clientsecrets.py View 1 3 chunks +16 lines, -13 lines 0 comments Download
M tests/data/client_secrets.json View 1 2 3 1 chunk +2 lines, -1 line 0 comments Download
M tests/test_discovery.py View 1 2 3 2 chunks +2 lines, -3 lines 0 comments Download
M tests/test_oauth2client.py View 1 2 3 4 5 6 7 22 chunks +135 lines, -54 lines 0 comments Download
M tests/test_oauth2client_appengine.py View 1 2 3 4 5 6 7 9 chunks +21 lines, -13 lines 0 comments Download
M tests/test_oauth2client_file.py View 1 2 3 4 5 6 7 8 9 10 1 chunk +1 line, -0 lines 0 comments Download
M tests/test_oauth2client_keyring.py View 1 2 3 2 chunks +2 lines, -2 lines 0 comments Download

Messages

Total messages: 16
dhermes
12 years, 6 months ago (2013-01-08 00:18:58 UTC) #1
dhermes
12 years, 6 months ago (2013-01-08 00:27:43 UTC) #2
jcgregorio_google
https://codereview.appspot.com/7033052/diff/24001/oauth2client/appengine.py File oauth2client/appengine.py (right): https://codereview.appspot.com/7033052/diff/24001/oauth2client/appengine.py#newcode187 oauth2client/appengine.py:187: def _revoke(self, unused_request): This isn't really needed is it, ...
12 years, 6 months ago (2013-01-08 18:16:48 UTC) #3
dhermes
12 years, 6 months ago (2013-01-08 18:20:37 UTC) #4
dhermes
https://codereview.appspot.com/7033052/diff/24001/oauth2client/appengine.py File oauth2client/appengine.py (right): https://codereview.appspot.com/7033052/diff/24001/oauth2client/appengine.py#newcode187 oauth2client/appengine.py:187: def _revoke(self, unused_request): Yeah. Brain fart, my bad. On ...
12 years, 6 months ago (2013-01-08 18:20:48 UTC) #5
jcgregorio_google
https://codereview.appspot.com/7033052/diff/21011/oauth2client/client.py File oauth2client/client.py (right): https://codereview.appspot.com/7033052/diff/21011/oauth2client/client.py#newcode877 oauth2client/client.py:877: raise NotImplementedError If I just got a plain NotImplementedError ...
12 years, 5 months ago (2013-01-09 14:05:36 UTC) #6
dhermes
https://codereview.appspot.com/7033052/diff/21011/oauth2client/client.py File oauth2client/client.py (right): https://codereview.appspot.com/7033052/diff/21011/oauth2client/client.py#newcode877 oauth2client/client.py:877: raise NotImplementedError I'm fine adding a message but didn't ...
12 years, 5 months ago (2013-01-09 16:59:25 UTC) #7
jcgregorio_google
https://codereview.appspot.com/7033052/diff/21011/oauth2client/client.py File oauth2client/client.py (right): https://codereview.appspot.com/7033052/diff/21011/oauth2client/client.py#newcode877 oauth2client/client.py:877: raise NotImplementedError OK, then find out if they can ...
12 years, 5 months ago (2013-01-09 17:05:47 UTC) #8
dhermes
https://codereview.appspot.com/7033052/diff/21011/oauth2client/client.py File oauth2client/client.py (right): https://codereview.appspot.com/7033052/diff/21011/oauth2client/client.py#newcode877 oauth2client/client.py:877: raise NotImplementedError Cool. Do you have a test account ...
12 years, 5 months ago (2013-01-09 17:06:59 UTC) #9
dhermes
I verified this works (using an access token for revocation). Updated the code minimally to ...
12 years, 5 months ago (2013-01-22 01:38:10 UTC) #10
jcgregorio_google
https://codereview.appspot.com/7033052/diff/38001/oauth2client/client.py File oauth2client/client.py (right): https://codereview.appspot.com/7033052/diff/38001/oauth2client/client.py#newcode712 oauth2client/client.py:712: def _do_revoke(self, http_request, token): Why does _do_revoke() exist? Why ...
12 years, 5 months ago (2013-01-23 14:14:59 UTC) #11
dhermes
https://codereview.appspot.com/7033052/diff/38001/oauth2client/client.py File oauth2client/client.py (right): https://codereview.appspot.com/7033052/diff/38001/oauth2client/client.py#newcode712 oauth2client/client.py:712: def _do_revoke(self, http_request, token): Because some credentials classes revoke ...
12 years, 5 months ago (2013-01-23 18:52:29 UTC) #12
dhermes
Bump
12 years, 5 months ago (2013-02-06 00:16:21 UTC) #13
jcgregorio_google
On 2013/02/06 00:16:21, dhermes wrote: > Bump LGTM
12 years, 5 months ago (2013-02-06 17:01:04 UTC) #14
dhermes
12 years, 5 months ago (2013-02-06 17:15:55 UTC) #15
dhermes
12 years, 5 months ago (2013-02-06 17:21:45 UTC) #16

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b