Made logging of failed validation more explicit.

diff --git a/cas_provider/views.py b/cas_provider/views.py
index 108bc38..eed2d64 100644 (file)
--- a/cas_provider/views.py
+++ b/cas_provider/views.py
@@ -135,6 +135,9 @@ def validate(request):
     if service is not None and ticket_string is not None:
         try:
             ticket = ServiceTicket.objects.get(ticket=ticket_string)
+        except ServiceTicket.DoesNotExist:
+            logger.exception("Tried to validate with an invalid ticket: %s / %s", ticket_string, service)
+        else:
             username = ticket.user.username
             ticket.delete()
 
@@ -143,9 +146,6 @@ def validate(request):
 
             return HttpResponse("yes\n%s\n%s" % (username, histories))
 
-        except Exception as e:
-            logger.exception("Got an exception!: %s"% e)
-
     return HttpResponse("no\n\n")