The wiki says, the login captcha is meant to prevent "Bruteforcing the passwords". In this case, I think, it is the wrong tool for the job. If someone makes an effort to get a password using brute force on the login form, the captcha won't stop him. It looks rather weak, so even standard OCR software should be able to read the text without applying some special algorithms. If you really want to prevent brute force attacks, you should deny the login (based on the IP address) every twenty failed attempts in a row for five minutes or something similar. As brute force involves testing a huge number of passwords, an attacker does not get very far with 240 passwords per hour, but a user who has forgotten his password and tries to recover it, might eventually succeed.
KeyDog wrote:you think its likely that they easily bypass via own openid provider?
I think, it is more likely that someone breaks the captcha. As noted above, it is not very strong, so breaking it does not require huge efforts. In addition, a captcha nowadays is a common obstacle, so attackers are already prepared to break it. OpenID, on the other hand, still is rather unknown, and I can't yet imagine an attacker, that is prepared to use it, for example by operating a provider that successfully authenticates every request to simulate a huge number of users.
KeyDog wrote:maybe options could be added to captcha ext
a. disable for openId registrations
b. disable for openId log ins....
That's the question. Should the antispam extension detect the presence of the OpenID extension, ask the OpenID extension if an OpenID request was received and in that case disable itself, or should the OpenID extension detect the presence of the antispam extension, and use its hooks to bypass the captcha? In my opinion, the latter option makes more sense (given that the hooks are at the right spots), but at the moment I won't implement it. Time is always short, so I have to concentrate on the important things, and as I personally would not use the antispam extension in the current state (weak captcha, captcha on login form), it can't be considered important. However, patches are welcome.
Edit: But I made one small change: My code removed too much markup responsible for displaying "required" at form fields, so for example the captcha input field does not look like it should. The new code will remove it only from the necessary fields. The change has already been comitted to the Git repository and will be in the next release.