[Findit] After login, redirect to the targeted page for GETs, but original page for others like POS…

appengine/findit/common/base_handler.py

 # Copyright 2014 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 import collections
 import json
 import logging
 
 from google.appengine.api import users
 import jinja2
@@ skipping 134 matching lines @@
       content_type = 'text/html'
 
     if cache_expiry is not None:
       self.response.headers['cache-control'] = (
           'max-age=%s, public' % cache_expiry)
     self.response.headers['Content-Type'] = content_type
     self.response.write(data)
 
   def GetLoginUrl(self):
     """Returns the login url."""
-    return users.create_login_url(self.request.referer or self.request.uri)
+    if self.request.method == 'GET':
+      # For GET, all parameters are included in the URL.
+      return users.create_login_url(self.request.url)
+    else:
+      # For others like POST, the parameters could be in the body and include
+      # file, etc. Thus return to the original page if available.
+      return users.create_login_url(self.request.referrer)
 
   def _Handle(self, handler_func):
     try:
       if not self._HasPermission():
         template = 'error.html'
         data = {
             'error_message':
                 ('Either not login or no permission. '
                  'Please login with your google.com account.'),
             'login_url': self.GetLoginUrl(),
@@ skipping 19 matching lines @@
       return_code = 500
       cache_expiry = None
 
     self._SendResponse(template, data, return_code, cache_expiry)
 
   def get(self):
     self._Handle(self.HandleGet)
 
   def post(self):
     self._Handle(self.HandlePost)