Opened 14 years ago

Closed 14 years ago

#63 closed defect (fixed)

Don't crash on users with hidden emails

Reported by: geofft Owned by:
Priority: normal Milestone: Sometime
Component: Default Version:
Keywords: Cc:
Size:

Description

We got two of these tracebacks today:

  File "/usr/lib/python2.7/site-packages/django/core/handlers/base.py", line 89, in get_response
    response = middleware_method(request)

  File "/usr/lib/python2.7/site-packages/django/contrib/auth/middleware.py", line 63, in process_request
    user = auth.authenticate(remote_user=username)

  File "/usr/lib/python2.7/site-packages/django/contrib/auth/__init__.py", line 55, in authenticate
    user = backend.authenticate(**credentials)

  File "/usr/lib/python2.7/site-packages/django/contrib/auth/backends.py", line 103, in authenticate
    user = self.configure_user(user)

  File "/mit/asa-db/Scripts/django/asa-db/asadb/mit/__init__.py", line 44, in configure_user
    user.email = result[0][1]['mail'][0]

KeyError: 'mail'

A proposed fix is in git://geofft.scripts.mit.edu/asa-db.git on the trac-63 branch.

Change History (1)

comment:1 Changed 14 years ago by adehnert

  • Resolution set to fixed
  • Status changed from new to closed

Fixed:

commit 7045dd8751322ffbf508cd7759bcc99bfef33f54
Author: Geoffrey Thomas <geofft@mit.edu>
Date:   Tue Jan 24 01:57:01 2012 -0500

    Django MIT plugin: Don't crash on users with hidden emails (ASA Trac: #63)
Note: See TracTickets for help on using tickets.