Changeset 169

Timestamp:

12/29/08 13:18:08 (1 year ago)

Author:

bettse

Message:

re #2850
This uses the already available SSO user info to fill in the user's name and email address anytime they autenticate to a project

Files:

• trac-plugins/accountmanagerplugin/trunk/acct_mgr/web_ui.py (modified) (2 diffs)
• trac-plugins/accountmanagerplugin/trunk/setup.py (modified) (1 diff)

trac-plugins/accountmanagerplugin/trunk/acct_mgr/web_ui.py

@@ -26 +26 @@
-from trac.util import Markup
-from trac.util.html import html, escape
+from trac.web.session import Session
-
-from api import AccountManager
@@ -390 +391 @@
-        user = mysso.getsessionuserinfo(req.incookie[self.sso_cookie].value)
-        if user:
+
+            #This code is based on the commented out code in _create_user.
+            #Creating a Session object and doing a save() on it would have been more elegant but didn't seem to work
+            req.args['name'] = user['userinfo']['firstname'] + ' ' + user['userinfo']['lastname']
+            req.args['email'] = user['userinfo']['email']
+
+            db = self.env.get_db_cnx()
+            cursor = db.cursor()
+
+            for key in ('name', 'email'):
+                value = req.args.get(key)
+                if not value:
+                    continue 
+                cursor.execute("UPDATE session_attribute SET value=%s "
+                               "WHERE name=%s AND sid=%s AND authenticated=1",
+                               (value, key, user['userinfo']['username'] ))
+                if not cursor.rowcount:
+                    cursor.execute("INSERT INTO session_attribute "
+                                   "(sid,authenticated,name,value) "
+                                   "VALUES (%s,1,%s,%s)",
+                                   (user['userinfo']['username'] , key, value))
+            db.commit()
+            self.log.info("New session saved for user %s" % req.args['name'] )
+        
+        
+        
-            req.environ['REMOTE_USER'] = user['userinfo']['username']
-            return req.environ['REMOTE_USER']

trac-plugins/accountmanagerplugin/trunk/setup.py

@@ -5 +5 @@
-setup(
-    name = 'TracAccountManager',
-1
-
-trac@matt-good.net
+2
+, Jose Cedeno, Eric Betts
+code@lists.oregonstate.edu
-    url = 'http://trac-hacks.org/wiki/AccountManagerPlugin',
-
+ with support for OSU SSO
-
-    license = '''