Make password encryption correctly handle non-ASCII strings.

Review Request #8453 — Created Oct. 11, 2016 and submitted — Latest diff uploaded

Information

Review Board
release-2.0.x
d89dea6...

Reviewers

It's rare for people to use non-ASCII passwords, but it does happen in some
cases. Most of our code deals with this correctly, but the password encryption
and decryption would fail. This change adds the necessary encode/decode steps.
If someone is using non-ASCII passwords with an encoding other than utf-8
they're out of luck.

  • Connected an SVN repository using a password containing unicode. Before this
    change, saving the form would fail. After, it works. I also checked the
    contents of the form field after saving it to verify that we were pulling the
    password back out correctly. I was also able to create a post-commit review
    request using this repository.
  • Ran unit tests.
    Loading...