-
-
Notifications
You must be signed in to change notification settings - Fork 7
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
additional option to perform matching and scoring with unicode support #16
Comments
Is it expected for ZXCVBN to raise an exception if provided a non-unicode string until this option is implemented? Currently I see:
This test is with emoji but I believe it holds for foreign language characters as well. |
@axelson yep that's a known issue and is happening because it handles ASCII only at this time. I am working on getting support for this. I believe non-ASII passwords are used by very less percentage of people given the lack of support on various different platforms. |
@techgaun any progress on this? |
@axelson I've added some support for this on master branch but need some time to do testing. Feel free to give master branch a try and see if that works in your case. I do need to spend some time to clean that up but I've not been finding time due to job and personal reasons for last few months. |
Okay, I'll take a look at the version in master when I get a chance |
No description provided.
The text was updated successfully, but these errors were encountered: