• 
      

    Fix handling of unicode in Active Directory group names.

    Review Request #6959 — Created Feb. 17, 2015 and submitted — Latest diff uploaded

    Information

    Review Board
    release-2.0.x
    435177c...

    Reviewers

    Our checking for the "required group" in the AD authentication backend wasn't
    correct in its handling of unicode. In the event that the required group had a
    non-ASCII character, we'd crash while trying to split the groups.

    I've changed it so that the text coming from AD is split using bytes instead of
    unicode strings, which should avoid any implicit coercion. I've also fixed the
    check to make sure we encode the required group to bytes before checking it
    against the array of bytes objects.

    Ran unit tests.