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
Comment #1
Benjamin Schrauwen commentedApplies to both 5.x and 6.x version.
Comment #2
dries commentedThis was fixed in the 1.6 version of both the D5 and D6 versions of the Mollom module.