Commit e5ca9618 by David Ormsbee

remove /login from tracking logs so we don't capture passwords

parent 79c1cf19
...@@ -74,8 +74,10 @@ def login_user(request, error=""): ...@@ -74,8 +74,10 @@ def login_user(request, error=""):
log.critical("Login failed - Could not create session. Is memcached running?") log.critical("Login failed - Could not create session. Is memcached running?")
log.exception(e) log.exception(e)
log.info("Login success - {0} ({1})".format(username, email))
return HttpResponse(json.dumps({'success':True})) return HttpResponse(json.dumps({'success':True}))
log.warning("Login failed - Account not active for user {0}".format(username))
return HttpResponse(json.dumps({'success':False, return HttpResponse(json.dumps({'success':False,
'error': 'Account not active. Check your e-mail.'})) 'error': 'Account not active. Check your e-mail.'}))
......
...@@ -5,10 +5,11 @@ from django.conf import settings ...@@ -5,10 +5,11 @@ from django.conf import settings
import views import views
class TrackMiddleware: class TrackMiddleware:
def process_request (self, request): def process_request(self, request):
try: try:
# We're already logging events # We're already logging events, and we don't want to capture user
if request.META['PATH_INFO'] == '/event': # names/passwords.
if request.META['PATH_INFO'] in ['/event', '/login']:
return return
event = { 'GET' : dict(request.GET), event = { 'GET' : dict(request.GET),
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment