Restore Py2.5 compatibility

[django-cas-provider.git] / cas_provider / views.py

diff --git a/cas_provider/views.py b/cas_provider/views.py
index 8a6e4bf..c9a44db 100644 (file)
--- a/cas_provider/views.py
+++ b/cas_provider/views.py
@@ -1,6 +1,6 @@
 from django.conf import settings
 from django.conf import settings

-from django.contrib.auth import login as auth_login, \
-    logout as auth_logout
+from django.contrib.auth import login as auth_login, logout as auth_logout
+from django.core.urlresolvers import get_callable

 from django.http import HttpResponse, HttpResponseRedirect
 from django.shortcuts import render_to_response
 from django.template import RequestContext
 from django.http import HttpResponse, HttpResponseRedirect
 from django.shortcuts import render_to_response
 from django.template import RequestContext


@@ -11,14 +11,16 @@ from models import ServiceTicket, LoginTicket
 __all__ = ['login', 'validate', 'logout', 'service_validate']
 
 
 __all__ = ['login', 'validate', 'logout', 'service_validate']
 
 

-INVALID_TICKET = 1
-INVALID_SERVICE = 2
-INVALID_REQUEST = 3
+INVALID_TICKET = 'INVALID_TICKET'
+INVALID_SERVICE = 'INVALID_SERVICE'
+INVALID_REQUEST = 'INVALID_REQUEST'
+INTERNAL_ERROR = 'INTERNAL_ERROR'

 
 ERROR_MESSAGES = (
     (INVALID_TICKET, u'The provided ticket is invalid.'),
     (INVALID_SERVICE, u'Service is invalid'),
     (INVALID_REQUEST, u'Not all required parameters were sent.'),
 
 ERROR_MESSAGES = (
     (INVALID_TICKET, u'The provided ticket is invalid.'),
     (INVALID_SERVICE, u'Service is invalid'),
     (INVALID_REQUEST, u'Not all required parameters were sent.'),

+    (INTERNAL_ERROR, u'An internal error occurred during ticket validation'),

 )
 
 
 )
 
 


@@ -63,19 +65,27 @@ def validate(request):
     service = request.GET.get('service', None)
     ticket_string = request.GET.get('ticket', None)
     if service is not None and ticket_string is not None:
     service = request.GET.get('service', None)
     ticket_string = request.GET.get('ticket', None)
     if service is not None and ticket_string is not None:

+        #renew = request.GET.get('renew', True)
+        #if not renew:
+        # TODO: check user SSO session

         try:
             ticket = ServiceTicket.objects.get(ticket=ticket_string)
         try:
             ticket = ServiceTicket.objects.get(ticket=ticket_string)

+            assert ticket.service == service

             username = ticket.user.username
             ticket.delete()
             username = ticket.user.username
             ticket.delete()

-            return HttpResponse("yes\r\n%s\r\n" % username)
+            return HttpResponse("yes\n%s\n" % username)

         except:
             pass
         except:
             pass

-    return HttpResponse("no\r\n\r\n")
+    return HttpResponse("no\n\n")

 
 
 
 

-def logout(request, template_name='cas/logout.html'):
+def logout(request, template_name='cas/logout.html', \
+                auto_redirect=settings.CAS_AUTO_REDIRECT_AFTER_LOGOUT):

     url = request.GET.get('url', None)
     url = request.GET.get('url', None)

-    auth_logout(request)
+    if request.user.is_authenticated():
+        auth_logout(request)
+        if url and auto_redirect:
+            return HttpResponseRedirect(url)

     return render_to_response(template_name, {'url': url}, \
                               context_instance=RequestContext(request))
 
     return render_to_response(template_name, {'url': url}, \
                               context_instance=RequestContext(request))
 


@@ -92,13 +102,13 @@ def service_validate(request):
     except ServiceTicket.DoesNotExist:
         return _cas2_error_response(INVALID_TICKET)
 
     except ServiceTicket.DoesNotExist:
         return _cas2_error_response(INVALID_TICKET)
 

-    if settings.CAS_CHECK_SERVICE and ticket.service != service:
+    if ticket.service != service:

         ticket.delete()
         return _cas2_error_response(INVALID_SERVICE)
 
         ticket.delete()
         return _cas2_error_response(INVALID_SERVICE)
 

-    username = ticket.user.username
+    user = ticket.user

     ticket.delete()
     ticket.delete()

-    return _cas2_sucess_response(username)
+    return _cas2_sucess_response(user)

 
 
 def _cas2_error_response(code):
 
 
 def _cas2_error_response(code):


@@ -112,9 +122,22 @@ def _cas2_error_response(code):
     }, mimetype='text/xml')
 
 
     }, mimetype='text/xml')
 
 

-def _cas2_sucess_response(username):
-    return HttpResponse(u'''<cas:serviceResponse xmlns:cas="http://www.yale.edu/tp/cas">
-        <cas:authenticationSuccess>
-            <cas:user>%(username)s</cas:user>
-        </cas:authenticationSuccess>
-    </cas:serviceResponse>''' % {'username': username}, mimetype='text/xml')
+def _cas2_sucess_response(user):
+    return HttpResponse(auth_success_response(user), mimetype='text/xml')
+
+
+def auth_success_response(user):
+    from attribute_formatters import CAS, NSMAP, etree
+
+    response = etree.Element(CAS + 'serviceResponse', nsmap=NSMAP)
+    auth_success = etree.SubElement(response, CAS + 'authenticationSuccess')
+    username = etree.SubElement(auth_success, CAS + 'user')
+    username.text = user.username
+
+    if settings.CAS_CUSTOM_ATTRIBUTES_CALLBACK:
+        callback = get_callable(settings.CAS_CUSTOM_ATTRIBUTES_CALLBACK)
+        attrs = callback(user)
+        if len(attrs) > 0:
+            formater = get_callable(settings.CAS_CUSTOM_ATTRIBUTES_FORMATER)
+            formater(auth_success, attrs)
+    return unicode(etree.tostring(response, encoding='utf-8'), 'utf-8')