avoid exception when email is undefined

Review Request #876 — Created May 26, 2009 and submitted

Information

Review Board SVN (deprecated)

Reviewers

If nor settings.LDAP_EMAIL_DOMAIN nor settings.LDAP_EMAIL_ATTRIBUTE are specified, the email variable is undefined and that causes an exception. The patch simply defines it to the empty string and logs a warning... not sure if it should be handled differently.

 
PB
Review request changed
Diff:

Revision 2 (+3)

Show changes

/trunk/reviewboard/accounts/backends.py
chipx86
  1. Thanks, committed as r1981.
  2. 
      
Loading...