django1.5 compatibility fixes

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

diff --git a/cas_provider/views.py b/cas_provider/views.py
index 2769e52..7aeaafb 100644 (file)
--- a/cas_provider/views.py
+++ b/cas_provider/views.py
@@ -1,18 +1,37 @@
-from lxml import etree
+import logging
+logger = logging.getLogger('cas_provider.views')
+import urllib
+
+import logging

 from urllib import urlencode
 import urllib2
 import urlparse
 from urllib import urlencode
 import urllib2
 import urlparse

+from functools import wraps
+
+from django.utils.decorators import available_attrs
+from django.views.decorators.debug import sensitive_post_parameters
+from django.views.decorators.cache import cache_control
+from django.utils.cache import patch_cache_control
+from django.views.decorators.csrf import csrf_protect
+
+from django.http import HttpResponse, HttpResponseRedirect

 from django.conf import settings
 from django.contrib.auth import login as auth_login, logout as auth_logout
 from django.core.urlresolvers import get_callable
 from django.conf import settings
 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.shortcuts import render_to_response
 from django.template import RequestContext

+from django.contrib.auth import authenticate
+from django.core.urlresolvers import reverse
+
+from lxml import etree

 from cas_provider.attribute_formatters import NSMAP, CAS
 from cas_provider.models import ProxyGrantingTicket, ProxyTicket
 from cas_provider.attribute_formatters import NSMAP, CAS
 from cas_provider.models import ProxyGrantingTicket, ProxyTicket

-from forms import LoginForm
-from models import ServiceTicket, LoginTicket
+from cas_provider.models import ServiceTicket

 
 

+from cas_provider.exceptions import SameEmailMismatchedPasswords
+from cas_provider.forms import LoginForm, MergeLoginForm
+
+from . import signals

 
 __all__ = ['login', 'validate', 'logout', 'service_validate']
 
 
 __all__ = ['login', 'validate', 'logout', 'service_validate']
 


@@ -29,49 +48,144 @@ ERROR_MESSAGES = (
     )
 
 
     )
 
 

-def login(request, template_name='cas/login.html',\
+logger = logging.getLogger(__name__)
+
+
+_never_cache = cache_control(no_cache=True, must_revalidate=True)
+
+
+def never_cache(view_func):
+    """
+    Decorator that adds headers to a response so that it will
+    never be cached.
+    """
+    @wraps(view_func, assigned=available_attrs(view_func))
+    def _wrapped_view_func(request, *args, **kwargs):
+        response = view_func(request, *args, **kwargs)
+        patch_cache_control(response, no_cache=True,
+                            must_revalidate=True, proxy_revalidate=True)
+        response['Pragma'] = 'no-cache'
+        return response
+    return _wrapped_view_func
+
+
+@sensitive_post_parameters()
+@csrf_protect
+@never_cache
+def login(request, template_name='cas/login.html',

           success_redirect=settings.LOGIN_REDIRECT_URL,
           success_redirect=settings.LOGIN_REDIRECT_URL,

-          warn_template_name='cas/warn.html',
-          form_class=LoginForm):
+          warn_template_name='cas/warn.html', **kwargs):
+    merge = kwargs.get('merge', False)
+    logging.debug('CAS Provider Login view. Method is %s, merge is %s, template is %s.',
+                  request.method, merge, template_name)
+

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

-    if request.user.is_authenticated():
-        if service is not None:
-            if request.GET.get('warn', False):
-                return render_to_response(warn_template_name, {
-                    'service': service,
-                    'warn': False
-                }, context_instance=RequestContext(request))
-            ticket = ServiceTicket.objects.create(service=service, user=request.user)
-            return HttpResponseRedirect(ticket.get_redirect_url())
-        else:
-            return HttpResponseRedirect(success_redirect)
+    if service is not None:
+        # Save the service on the session, for later use if we end up
+        # in one of the more complicated workflows.
+        request.session['service'] = service
+
+    user = request.user
+
+    errors = []
+

     if request.method == 'POST':
     if request.method == 'POST':

-        form = form_class(data=request.POST, request=request)
+        if merge:
+            form = MergeLoginForm(request.POST, request=request)
+        else:
+            form = LoginForm(request.POST, request=request)
+

         if form.is_valid():
         if form.is_valid():

-            user = form.get_user()
-            auth_login(request, user)
-            service = form.cleaned_data.get('service')
-            if service is not None:
+            service = form.cleaned_data.get('service', None)
+            try:
+                auth_args = dict(username=form.cleaned_data['email'],
+                                 password=form.cleaned_data['password'])
+                if merge:
+                    # We only want to send the merge argument if it's
+                    # True. If it it's False, we want it to propagate
+                    # through the auth backends properly.
+                    auth_args['merge'] = merge
+                user = authenticate(**auth_args)
+            except SameEmailMismatchedPasswords:
+                # Need to merge the accounts?
+                if merge:
+                    # We shouldn't get here...
+                    raise
+                else:
+                    base_url = reverse('cas_provider_merge')
+                    args = dict(
+                        success_redirect=success_redirect,
+                        email=form.cleaned_data['email'],
+                        )
+                    if service is not None:
+                        args['service'] = service
+                    args = urllib.urlencode(args)
+
+                    url = '%s?%s' % (base_url, args)
+                    logging.debug('Redirecting to %s', url)
+                    return HttpResponseRedirect(url)
+
+            if user is None:
+                errors.append('Incorrect username and/or password.')
+            else:
+                if user.is_active:
+                    auth_login(request, user)
+
+    else:  # Not a POST...
+        if merge:
+            form = MergeLoginForm(initial={'service': service, 'email': request.GET.get('email')})
+        else:
+            form = LoginForm(initial={'service': service})
+
+    if user is not None and user.is_authenticated():
+        # We have an authenticated user.
+        if not user.is_active:
+            errors.append('This account is disabled.')
+        else:
+            # Send the on_cas_login signal. If we get an HttpResponse, return that.
+            for receiver, response in signals.on_cas_login.send(sender=login, request=request, **kwargs):
+                if isinstance(response, HttpResponse):
+                    return response
+
+            if service is None:
+                # Try and pull the service off the session
+                service = request.session.pop('service', service)
+
+            signals.on_cas_login_success.send(sender=login, request=request,
+                                              service=service, **kwargs)
+
+            if service is None:
+                # Normal internal success redirection.
+                logging.debug('Redirecting to %s', success_redirect)
+                return HttpResponseRedirect(success_redirect)
+            else:
+                if request.GET.get('warn', False):
+                    return render_to_response(warn_template_name, {
+                        'service': service,
+                        'warn': False
+                    }, context_instance=RequestContext(request))
+
+                # Create a service ticket and redirect to the service.

                 ticket = ServiceTicket.objects.create(service=service, user=user)
                 ticket = ServiceTicket.objects.create(service=service, user=user)

-                success_redirect = ticket.get_redirect_url()
-            return HttpResponseRedirect(success_redirect)
-    else:
-        form = form_class(request=request, initial={
-            'service': service,
-            'lt': LoginTicket.objects.create()
-        })
-    if hasattr(request, 'session') and hasattr(request.session, 'set_test_cookie'):
-        request.session.set_test_cookie()
-    return render_to_response(template_name, {
-        'form': form,
-        'errors': form.get_errors() if hasattr(form, 'get_errors') else None,
-        }, context_instance=RequestContext(request))
+                if 'service' in request.session:
+                    # Don't need this any more.
+                    del request.session['service']
+
+                url = ticket.get_redirect_url()
+                logging.debug('Redirecting to %s', url)
+                return HttpResponseRedirect(url)
+
+    logging.debug('Rendering response on %s, merge is %s', template_name, merge)
+    return render_to_response(template_name, {'form': form, 'errors': errors}, context_instance=RequestContext(request))

 
 
 
 

+@never_cache

 def validate(request):
 def validate(request):

-    """Validate ticket via CAS v.1 protocol"""
+    """Validate ticket via CAS v.1 protocol
+    """

     service = request.GET.get('service', None)
     ticket_string = request.GET.get('ticket', None)
     service = request.GET.get('service', None)
     ticket_string = request.GET.get('ticket', None)

+    logger.info('Validating ticket %s for %s', ticket_string, service)

     if service is not None and ticket_string is not None:
         #renew = request.GET.get('renew', True)
         #if not renew:
     if service is not None and ticket_string is not None:
         #renew = request.GET.get('renew', True)
         #if not renew:


@@ -79,13 +193,24 @@ def validate(request):
         try:
             ticket = ServiceTicket.objects.get(ticket=ticket_string)
             assert ticket.service == service
         try:
             ticket = ServiceTicket.objects.get(ticket=ticket_string)
             assert ticket.service == service

+        except ServiceTicket.DoesNotExist:
+            logger.exception("Tried to validate with an invalid ticket %s for %s", ticket_string, service)
+        except Exception as e:
+            logger.exception('Got an exception: %s', e)
+        else:

             username = ticket.user.username
             username = ticket.user.username

-            return HttpResponse("yes\n%s\n" % username)
-        except:
-            pass
+            ticket.delete()
+
+            results = signals.on_cas_collect_histories.send(sender=validate, for_user=ticket.user)
+            histories = '\n'.join('\n'.join(rs) for rc, rs in results)
+            logger.info('Validated %s %s', username, "(also %s)" % histories if histories else '')
+            return HttpResponse("yes\n%s\n%s" % (username, histories))
+
+    logger.info('Validation failed.')

     return HttpResponse("no\n\n")
 
 
     return HttpResponse("no\n\n")
 
 

+@never_cache

 def logout(request, template_name='cas/logout.html',
            auto_redirect=settings.CAS_AUTO_REDIRECT_AFTER_LOGOUT):
     url = request.GET.get('url', None)
 def logout(request, template_name='cas/logout.html',
            auto_redirect=settings.CAS_AUTO_REDIRECT_AFTER_LOGOUT):
     url = request.GET.get('url', None)


@@ -99,20 +224,16 @@ def logout(request, template_name='cas/logout.html',
         context_instance=RequestContext(request))
 
 
         context_instance=RequestContext(request))
 
 

+@never_cache

 def proxy(request):
     targetService = request.GET['targetService']
 def proxy(request):
     targetService = request.GET['targetService']

-    pgtiou = request.GET['pgt']
+    pgt_id = request.GET['pgt']

 
     try:
 
     try:

-        proxyGrantingTicket = ProxyGrantingTicket.objects.get(pgtiou=pgtiou)
+        proxyGrantingTicket = ProxyGrantingTicket.objects.get(ticket=pgt_id)

     except ProxyGrantingTicket.DoesNotExist:
         return _cas2_error_response(INVALID_TICKET)
 
     except ProxyGrantingTicket.DoesNotExist:
         return _cas2_error_response(INVALID_TICKET)
 

-    if not proxyGrantingTicket.targetService == targetService:
-        return _cas2_error_response(INVALID_SERVICE,
-            "The PGT was issued for %(original)s but the PT was requested for %(but)s" % dict(
-                original=proxyGrantingTicket.targetService, but=targetService))
-

     pt = ProxyTicket.objects.create(proxyGrantingTicket=proxyGrantingTicket,
         user=proxyGrantingTicket.serviceTicket.user,
         service=targetService)
     pt = ProxyTicket.objects.create(proxyGrantingTicket=proxyGrantingTicket,
         user=proxyGrantingTicket.serviceTicket.user,
         service=targetService)


@@ -135,7 +256,10 @@ def ticket_validate(service, ticket_string, pgtUrl):
     except ServiceTicket.DoesNotExist:
         return _cas2_error_response(INVALID_TICKET)
 
     except ServiceTicket.DoesNotExist:
         return _cas2_error_response(INVALID_TICKET)
 

-    if ticket.service != service:
+    ticketUrl = urlparse.urlparse(ticket.service)
+    serviceUrl = urlparse.urlparse(service)
+
+    if not(ticketUrl.hostname == serviceUrl.hostname and ticketUrl.path == serviceUrl.path and ticketUrl.port == serviceUrl.port):

         return _cas2_error_response(INVALID_SERVICE)
 
     pgtIouId = None
         return _cas2_error_response(INVALID_SERVICE)
 
     pgtIouId = None


@@ -157,9 +281,11 @@ def ticket_validate(service, ticket_string, pgtUrl):
                     pgt = None
 
     user = ticket.user
                     pgt = None
 
     user = ticket.user

+    ticket.delete()

     return _cas2_sucess_response(user, pgtIouId, proxies)
 
 
     return _cas2_sucess_response(user, pgtIouId, proxies)
 
 

+@never_cache

 def service_validate(request):
     """Validate ticket via CAS v.2 protocol"""
     service = request.GET.get('service', None)
 def service_validate(request):
     """Validate ticket via CAS v.2 protocol"""
     service = request.GET.get('service', None)


@@ -171,6 +297,7 @@ def service_validate(request):
         return ticket_validate(service, ticket_string, pgtUrl)
 
 
         return ticket_validate(service, ticket_string, pgtUrl)
 
 

+@never_cache

 def proxy_validate(request):
     """Validate ticket via CAS v.2 protocol"""
     service = request.GET.get('service', None)
 def proxy_validate(request):
     """Validate ticket via CAS v.2 protocol"""
     service = request.GET.get('service', None)


@@ -196,14 +323,16 @@ def generate_proxy_granting_ticket(pgt_url, ticket):
     query = dict(urlparse.parse_qsl(uri[4]))
     query.update(params)
 
     query = dict(urlparse.parse_qsl(uri[4]))
     query.update(params)
 

-    uri[4] = urlencode(query)
+    uri[3] = urlencode(query)

 
     try:
         response = urllib2.urlopen(urlparse.urlunsplit(uri))
 
     try:
         response = urllib2.urlopen(urlparse.urlunsplit(uri))

-    except urllib2.HTTPError, e:
+    except urllib2.HTTPError as e:

         if not e.code in proxy_callback_good_status:
         if not e.code in proxy_callback_good_status:

+            logger.debug('Checking Proxy Callback URL {} returned {}. Not issuing PGT.'.format(uri, e.code))

             return
             return

-    except urllib2.URLError, e:
+    except urllib2.URLError as e:
+        logger.debug('Checking Proxy Callback URL {} raised URLError. Not issuing PGT.'.format(uri))

         return
 
     pgt.save()
         return
 
     pgt.save()


@@ -243,12 +372,21 @@ def auth_success_response(user, pgt, proxies):
     username = etree.SubElement(auth_success, CAS + 'user')
     username.text = user.username
 
     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)
+    attrs = {}
+    for receiver, custom in signals.cas_collect_custom_attributes.send(sender=auth_success_response, user=user):
+        if custom:
+            attrs.update(custom)
+
+    identifiers = [i for sr, rr in signals.on_cas_collect_histories.send(sender=validate, for_user=user)
+                   for i in rr]
+
+    if identifiers:
+        # Singular `identifier`, as that is the name of the element tag(s).
+        attrs['identifier'] = identifiers
+
+    if attrs:
+        formatter = get_callable(settings.CAS_CUSTOM_ATTRIBUTES_FORMATER)
+        formatter(auth_success, attrs)

 
     if pgt:
         pgtElement = etree.SubElement(auth_success, CAS + 'proxyGrantingTicket')
 
     if pgt:
         pgtElement = etree.SubElement(auth_success, CAS + 'proxyGrantingTicket')