• 
      

    Allow usernames with '@' in the admin backend

    Review Request #2124 — Created Feb. 15, 2011 and submitted — Latest diff uploaded

    Information

    Review Board

    Reviewers

    Allow usernames with '@' in the admin backend
    
    It is an usual situation for LDAP servers use the mail attribute
    for authentication. If ReviewBoard is set up to authenticate using
    an email attribute, login is successful and the user data is
    correctly populated. Nevertheless, it is not possible to change the user
    field as the username is considered incorrect.
    
    This commit relaxes the restriction in the admin backend to allow for
    the '@' character in usernames. There is no need to perform a similar
    change in the frontend code as LDAP registrations do not follow the same path
    as basic registrations.
    Performed a LDAP registration, and them modified the account in the admin backend.