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

Sinan Erdem’s picture

I am also experiencing this behavior on my sites...

xamount’s picture

subscribe

dave reid’s picture

Version: 6.x-1.10 » 6.x-1.x-dev
Assigned: Unassigned » dave reid

I can confirm this and will have a fix with a test to back it up shortly.

dave reid’s picture

Assigned: dave reid » Unassigned

I'll have to see if this still is a problem with the latest code actually.

sun’s picture

Status: Active » Fixed

Let's mark this as fixed until someone can actually replicate the problem with the latest code.

infojunkie’s picture

Version: 6.x-1.x-dev » 6.x-1.9

I can confirm that the erroneous behaviour above does not occur anymore for 6.x-1.9.

infojunkie’s picture

Version: 6.x-1.9 » 6.x-1.10

Sorry, 6.x-1.10.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.