The Mollom API can be pinged to determine if keys are correctly configured and if Mollom is able to communicate back to the web site. While the first part is already covered within the module, the return communication is not something that is currently verifiable. This patch adds the ability to ping Mollom and receive a pong callback. This does require that Mollom be set to log all activity (otherwise a successful pong callback is not logged as it isn't high severity). I'm comfortable with this given that it is only a troubleshooting technique.

CommentFileSizeAuthor
#14 mollom.ping_.backport.14.patch14.06 KBeshta
#12 mollom.ping_.backport.12.patch14.08 KBeshta
#10 mollom.ping_.backport.10.patch14.09 KBeshta
#8 mollom.ping_.backport.8.patch14.09 KBeshta
#5 mollom.ping_.5.patch13.69 KBeshta
#3 mollom.ping_.3.patch13.42 KBeshta
#1 mollom.ping_.1.patch11.29 KBeshta
mollom.ping_.0.patch6.54 KBeshta

Comments

eshta’s picture

eshta commented
StatusFileSize
new mollom.ping_.1.patch11.29 KB

Added tests and cleaned up UI.

Status: Needs review » Needs work

The last submitted patch, 1: mollom.ping_.1.patch, failed testing.

eshta’s picture

eshta commented
Status: Needs work » Needs review
StatusFileSize
new mollom.ping_.3.patch13.42 KB

Silly simpletest.....

eshta’s picture

eshta commented 
+++ b/tests/mollom.test
@@ -1372,6 +1372,30 @@ class MollomResponseTestCase extends MollomWebTestCase {
+    */

Oops - need to take this out still.

eshta’s picture

eshta commented
StatusFileSize
new mollom.ping_.5.patch13.69 KB

Now with unused code removed and PHP library class updated.

eshta’s picture

eshta commented
Issue summary: View changes

  • eshta committed e70d7e8 on 7.x-2.x 
    Issue #2389829 by eshta: Add ability to ping the Mollom API to test...
eshta’s picture

eshta commented
Version: 7.x-2.x-dev » 6.x-2.x-dev
StatusFileSize
new mollom.ping_.backport.8.patch14.09 KB

And now on to back-porting. My local d6 tests are having problems so let's see what happens with the test bots.

Status: Needs review » Needs work

The last submitted patch, 8: mollom.ping_.backport.8.patch, failed testing.

eshta’s picture

eshta commented
Status: Needs work » Needs review
StatusFileSize
new mollom.ping_.backport.10.patch14.09 KB

Go bots go.

Status: Needs review » Needs work

The last submitted patch, 10: mollom.ping_.backport.10.patch, failed testing.

eshta’s picture

eshta commented
Status: Needs work » Needs review
StatusFileSize
new mollom.ping_.backport.12.patch14.08 KB

Again.

Status: Needs review » Needs work

The last submitted patch, 12: mollom.ping_.backport.12.patch, failed testing.

eshta’s picture

eshta commented
Status: Needs work » Needs review
StatusFileSize
new mollom.ping_.backport.14.patch14.06 KB

And again....

  • eshta committed 96a5c67 on 6.x-2.x 
    Issue #2389829 by eshta: Add ability to ping the Mollom API to test...

  • eshta committed 9bf90fb on 6.x-2.x 
    Issue ##2389829 by eshta: Handle response codes returned as an array or...

  • eshta committed ad5a229 on 7.x-2.x 
    Issue ##2389829 by eshta: Handle response codes returned as an array or...
eshta’s picture

eshta commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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

eshta’s picture

eshta commented
Version: 6.x-2.x-dev » 8.x-1.x-dev
Status: Closed (fixed) » Patch (to be ported)
Issue tags: +needs port to Drupal 8
Related issues: +#2050547: Port Mollom to Drupal 8
eshta’s picture

eshta commented
Status: Patch (to be ported) » Closed (won't fix)

This is outdated as the ping callback has been removed from the D7 and D6 modules.