In the mollom function, which performs the actual call to Mollom and handles the client-side load balancing, a foreach-construct loops over all available servers. In this loop it is however possible that a REFRESH error is received which forces the module to reload its server list. Because the foreach operates on a copy of the servers array, this server list update is not noticed which could lead to a user-side error if the server list gets exhausted without correct handling of the post. If the user however re-posts the form, the server list is updated and everything goes as planned.

This inconvenience could be solved by using an iteration construction that does not work on a copy of the array. The iterator should also be reset if the REFRESH error is received.

Comments

Benjamin Schrauwen’s picture

Version: 5.x-1.4 » 6.x-1.x-dev

Applies to both 5.x and 6.x version.

dries’s picture

Status: Active » Fixed

This was fixed in the 1.6 version of both the D5 and D6 versions of the Mollom module.

Status: Fixed » Closed (fixed)

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