X-Git-Url: http://git.onelab.eu/?a=blobdiff_plain;f=auth%2Fmanifoldbackend.py;h=c7ac7faacc1336db4caacf3993786de80294c5d0;hb=485d5ae5ddd76213a6ba7d80a7a6d5048786b804;hp=2b1bde59a93258b43d43a56582d7bc5944ee83f1;hpb=57c5bcefb1eac66d1db27db28a0c6d3b3b52a407;p=unfold.git

diff --git a/auth/manifoldbackend.py b/auth/manifoldbackend.py
index 2b1bde59..c7ac7faa 100644
--- a/auth/manifoldbackend.py
+++ b/auth/manifoldbackend.py
@@ -2,7 +2,10 @@ import time
 
 from django.contrib.auth.models import User
 
-from manifold.manifoldapi import ManifoldAPI, ManifoldResult
+from manifoldapi.manifoldapi    import ManifoldAPI, ManifoldException, ManifoldResult
+from manifold.core.query        import Query
+
+from myslice.settings import config, logger, DEBUG
 
 # Name my backend 'ManifoldBackend'
 class ManifoldBackend:
@@ -12,6 +15,8 @@ class ManifoldBackend:
     def authenticate(self, token=None):
         if not token:
             return None
+        
+        person = {}
 
         try:
             username = token['username']
@@ -20,32 +25,36 @@ class ManifoldBackend:
 
             auth = {'AuthMethod': 'password', 'Username': username, 'AuthString': password}
             api = ManifoldAPI(auth)
-            # Authenticate user and get session key
-            session_result = api.GetSession()
-            session = session_result.ok_value()
-            if not session:
-                print "GetSession failed",session_result.error()
+            sessions_result = api.forward(Query.create('local:session').to_dict())
+            sessions = sessions_result.ok_value()
+            if not sessions:
+                logger.error("GetSession failed", sessions_result.error())
                 return
+            session = sessions[0]
+            logger.debug("SESSION : %s" % session)
             
-            print 'DEALING with session',session
-            #self.session = session
-            # Change GetSession() at some point to return expires as well
-            expires = time.time() + (24 * 60 * 60)
-
             # Change to session authentication
-            api.auth = {'AuthMethod': 'session', 'session': session}
+            api.auth = {'AuthMethod': 'session', 'session': session['session']}
             self.api = api
 
             # Get account details
-            persons_result = api.GetPersons(auth)
+            # the new API would expect Get('local:user') instead
+            persons_result = api.forward(Query.get('local:user').to_dict())
             persons = persons_result.ok_value()
             if not persons:
-                print "GetPersons failed",persons_result.error()
+                logger.error("GetPersons failed",persons_result.error())
                 return
             person = persons[0]
+            logger.debug("PERSON : %s" % person)
+            #logger.info("%s %s <%s> logged in" % (person['config']['first_name'], person['config']['last_name'], person['config']['email']))
 
-            request.session['manifold'] = {'auth': api.auth, 'person': person, 'expires': expires}
-        except:
+            request.session['manifold'] = {'auth': api.auth, 'person': person, 'expires': session['expires']}
+        except ManifoldException, e:
+            logger.error("Manifold Auth Backend: %s" % e.manifold_result)
+        except Exception, e:
+            logger.error("Manifold Auth Backend: %s" % e)
+            #import traceback
+            #traceback.print_exc()
             return None
 
         try:
@@ -54,9 +63,13 @@ class ManifoldBackend:
         except User.DoesNotExist:
             # Create a user in Django's local database
             user = User.objects.create_user(username, username, 'passworddoesntmatter')
-            user.first_name = person['first_name']
-            user.last_name = person['last_name']
             user.email = person['email']
+
+        if 'firstname' in person:
+            user.first_name = person['firstname']
+        if 'lastname' in person:
+            user.last_name = person['lastname']
+
         return user
 
     # Required for your backend to work properly - unchanged in most scenarios