Had several reports of people being confused about the watchdog message "Mollom ID x has been tampered with: it was generated for a user with sid y, but is used by a user with sid z." How can we make this more clear? Should we add something to the module handbook pages about it?

Comments

sun’s picture

Version: 6.x-1.x-dev » 7.x-1.x-dev
Status: Active » Needs review
StatusFileSize
new2.76 KB

Not only the message strings need work, but also the severity levels.

Some of them are clear indicators for malicious actions performed on a site. Mollom should at least warn about them.

dries’s picture

Status: Needs review » Reviewed & tested by the community

These changes look good to me, although I'm a bit afraid they are going to increase our support burden. Already, we get people e-mailing asking us what to do about those problems. When seeing those messages, people don't always realize that Mollom is doing its job (versus Mollom being broken).

sun’s picture

Status: Reviewed & tested by the community » Needs review
StatusFileSize
new2.92 KB

Alternative patch. Some more discussion may prevent support requests in the first place. Doable, I think.

dries’s picture

Version: 7.x-1.x-dev » 6.x-1.x-dev
Status: Needs review » Needs work

Looked at the latest patch with a fresh pair of eyes (good morning!), and decided it is an improvement. Committed the patch in #3 to CVS HEAD (Drupal 7).

sun’s picture

Status: Needs work » Reviewed & tested by the community
StatusFileSize
new2.84 KB

Same patch for D6.

dries’s picture

Status: Reviewed & tested by the community » Fixed

Great. Committed to DRUPAL-6--1. Thanks sun.

Status: Fixed » Closed (fixed)

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

  • Commit c5be677 on master, fai6, 8.x-2.x, fbajs, actions by Dries:
    - Patch #637046 by sun: improved some watchdog messages.
    
    

  • Commit c5be677 on master, fai6, 8.x-2.x, fbajs, actions by Dries:
    - Patch #637046 by sun: improved some watchdog messages.