During the Mollom outage a few days ago, we decided to test our site's ability to continue operating. We set the Mollom fallback strategy to "Leave all forms unprotected and accept all submissions".
Here's a scenario that fails: by protecting the user registration form using CAPTCHA, attempting to register a user while Mollom servers are unreachable produces a form validation error: "The CAPTCHA field is required."
To reproduce it now, open mollom.inc and in _mollom_retrieve_server_list(), insert the line return array(); right before the line return $result;. This simulates that Mollom servers are not responding.
To fix this, I edited mollom_expand_element() such that the function avoids calling mollom_validate_captcha() if $mollom_state['#session_id'] is empty. I'm not sure this is 100% correct though because I found a commented section of code that also invalidated the form when the session ID was empty.
Comments
Comment #1
Sinan Erdem commentedI am also experiencing this behavior on my sites...
Comment #2
xamountsubscribe
Comment #3
dave reidI can confirm this and will have a fix with a test to back it up shortly.
Comment #4
dave reidI'll have to see if this still is a problem with the latest code actually.
Comment #5
sunLet's mark this as fixed until someone can actually replicate the problem with the latest code.
Comment #6
infojunkieI can confirm that the erroneous behaviour above does not occur anymore for 6.x-1.9.
Comment #7
infojunkieSorry, 6.x-1.10.