[copr] master: Don't ask FAS for details of already registered users (7611d15)

bkabrda at fedoraproject.org bkabrda at fedoraproject.org
Fri Jan 25 12:27:53 UTC 2013


Repository : http://git.fedorahosted.org/cgit/copr.git

On branch  : master

>---------------------------------------------------------------

commit 7611d155d56ed0ae0d5b1ccd53d743babf044107
Author: Bohuslav Kabrda <bkabrda at redhat.com>
Date:   Fri Jan 25 13:27:24 2013 +0100

    Don't ask FAS for details of already registered users


>---------------------------------------------------------------

 coprs_frontend/coprs/views/misc.py |    5 ++++-
 1 files changed, 4 insertions(+), 1 deletions(-)

diff --git a/coprs_frontend/coprs/views/misc.py b/coprs_frontend/coprs/views/misc.py
index e681b48..e10b07f 100644
--- a/coprs_frontend/coprs/views/misc.py
+++ b/coprs_frontend/coprs/views/misc.py
@@ -48,7 +48,10 @@ def login():
         if fasusername and ((app.config['USE_ALLOWED_USERS'] \
             and fasusername in app.config['ALLOWED_USERS']) \
             or not app.config['USE_ALLOWED_USERS']):
-            return oid.try_login('http://{0}.id.fedoraproject.org/'.format(fasusername), ask_for = ["email"])
+            ask_for = []
+            if not models.User.query.filter(models.User.openid_name==models.User.openidize_name(fasusername)).first():
+                ask_for.append('email')
+            return oid.try_login('http://{0}.id.fedoraproject.org/'.format(fasusername), ask_for=ask_for)
         else:
             return flask.render_template('login.html',
                             error='User "{0}" is not allowed'.format(



More information about the copr-devel mailing list