Skip to content
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

Hint that "testPhrase" method should be used #96

Open
alex-gevatter opened this issue Dec 8, 2021 · 0 comments
Open

Hint that "testPhrase" method should be used #96

alex-gevatter opened this issue Dec 8, 2021 · 0 comments

Comments

@alex-gevatter
Copy link

alex-gevatter commented Dec 8, 2021

I did not use the provided "testPhrase" method for the validation of the user's CAPTCHA input string because I thought that this would be exaggerated only to compare two strings. This sometimes led to unsolvable CAPTCHAS, since one of the randomly selected fonts contains only uppercase letters. For example, if the generated phrase is "AbCdE", it becomes "ABCDE" in the CAPTCHA image. If you use the "testPhrase" method to validate the user input, this is not a problem. But if you do the validation yourself, as I did, you can end up with intractable CAPTCHAS.
The documentation states: "You can compare the phrase with user input". I think it would be better to write "must" or "should" here instead of "can" and explain why. A hint that a self-implemented case-sensitive string comparison leads to serious problems due to the upper-case font used would not be bad at this point either.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant