'CAPTCHA required' even though Mollom servers don't respond

kratib - September 22, 2009 - 22:16
Project:Mollom
Version:6.x-1.x-dev
Component:Code
Category:bug report
Priority:normal
Assigned:Dave Reid
Status:active
Description

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.

#1

etcetera9 - October 11, 2009 - 11:34

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

#2

xamount - October 20, 2009 - 04:27

subscribe

#3

Dave Reid - October 22, 2009 - 04:01
Version:6.x-1.10» 6.x-1.x-dev
Assigned to:Anonymous» Dave Reid

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

 
 

Drupal is a registered trademark of Dries Buytaert.