X-Git-Url: https://git.mdrn.pl/django-cas-provider.git/blobdiff_plain/09adf9da3df6fc5807c5b1710f967b39c101ba6c..3043bb2f2f00a94f8499c00b54b446d2dbd07a98:/cas_provider/views.py diff --git a/cas_provider/views.py b/cas_provider/views.py index 2769e52..3f2607f 100644 --- a/cas_provider/views.py +++ b/cas_provider/views.py @@ -1,18 +1,30 @@ -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 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.http import HttpResponse, HttpResponseRedirect 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 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'] @@ -29,49 +41,119 @@ ERROR_MESSAGES = ( ) -def login(request, template_name='cas/login.html',\ +logger = logging.getLogger(__name__) + + +def login(request, template_name='cas/login.html', 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) - 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': - 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(): - 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) + + 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) - 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)) 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) + 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: @@ -79,12 +161,22 @@ def validate(request): 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 - return HttpResponse("yes\n%s\n" % username) - except: - pass - return HttpResponse("no\n\n") + ticket.delete() + results = signals.on_cas_collect_histories.send(sender=validate, for_email=ticket.user.email) + 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") + def logout(request, template_name='cas/logout.html', auto_redirect=settings.CAS_AUTO_REDIRECT_AFTER_LOGOUT): @@ -101,18 +193,13 @@ def logout(request, template_name='cas/logout.html', def proxy(request): targetService = request.GET['targetService'] - pgtiou = request.GET['pgt'] + pgt_id = request.GET['pgt'] try: - proxyGrantingTicket = ProxyGrantingTicket.objects.get(pgtiou=pgtiou) + proxyGrantingTicket = ProxyGrantingTicket.objects.get(ticket=pgt_id) 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) @@ -135,7 +222,10 @@ def ticket_validate(service, ticket_string, pgtUrl): 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 @@ -157,6 +247,7 @@ def ticket_validate(service, ticket_string, pgtUrl): pgt = None user = ticket.user + ticket.delete() return _cas2_sucess_response(user, pgtIouId, proxies) @@ -196,14 +287,16 @@ def generate_proxy_granting_ticket(pgt_url, ticket): 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)) - except urllib2.HTTPError, e: + except urllib2.HTTPError as e: if not e.code in proxy_callback_good_status: + logger.debug('Checking Proxy Callback URL {} returned {}. Not issuing PGT.'.format(uri, e.code)) 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() @@ -247,8 +340,8 @@ def auth_success_response(user, pgt, proxies): 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) + formatter = get_callable(settings.CAS_CUSTOM_ATTRIBUTES_FORMATER) + formatter(auth_success, attrs) if pgt: pgtElement = etree.SubElement(auth_success, CAS + 'proxyGrantingTicket')