Page MenuHomePhabricator

Add logging/readonly/monitor mode
Open, MediumPublic

Description

It'd be useful for testing and stuff if we can make StopForum spam almost readonly (not blocking), but actually logging what IPs would've been used

Saves doing some of the analytics manually

Event Timeline

Reedy created this task.Jul 23 2019, 2:29 PM
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptJul 23 2019, 2:29 PM
Reedy renamed this task from Add logging/readyonly mode to Add logging/readonly/monitor mode.Jul 23 2019, 2:32 PM
sbassett triaged this task as Medium priority.Jul 23 2019, 7:31 PM

Could we piggy-back on what's there?

Reedy added a comment.Jul 24 2019, 6:17 AM

Could we piggy-back on what's there?

I think so yeah. Looks like the logging we want is already there, just to neuter the functionality. We might want to keep the output of the whitelisting (so they're not false positives) and then shove an if before the $result = [ 'stopforumspam-blocked', $ip ]; line with the config var, and then return true