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

Issue 4515195: Drop use of @login_decorator since it can only be used on GET requests. (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
14 years, 1 month ago by jcgregorio_google
Modified:
14 years ago
Reviewers:
jcgregorio_google, ade
CC:
google-api-python-client_googlegroups.com
Visibility:
Public.

Patch Set 1 #

Patch Set 2 : fixed 80 chars #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+10 lines, -4 lines) Patch
M oauth2client/appengine.py View 1 2 chunks +10 lines, -4 lines 1 comment Download

Messages

Total messages: 2
jcgregorio_google
14 years, 1 month ago (2011-06-06 18:20:41 UTC) #1
ade
14 years ago (2011-06-12 16:04:38 UTC) #2
http://codereview.appspot.com/4515195/diff/2001/oauth2client/appengine.py
File oauth2client/appengine.py (left):

http://codereview.appspot.com/4515195/diff/2001/oauth2client/appengine.py#old...
oauth2client/appengine.py:224: def check_oauth(request_handler, *args):
Can you add a comment to the sections of duplicated code explaining that we're
not using the login_required decorator because we need to support POST requests.
Sign in to reply to this message.

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