Hi Bryan,
I pulled the RC2, played with it a bit, and added in the production worthy changes I'd made to RC1. Patch attached.
- patch -p1 -i httpbl.module.patch
- - (against current httpbl-6.x-2.0-rc2)
- Incorporates the other changes I made wrt 'httpbl_blacklist_comment_spammer'
- - fixed whitespace around the IF for the same
- Added more blacklist_threshold_options (user couldn't lower it below 50)
- Added Description to Advanced form (modify on line 307 as desired)
I've got it currently running on two personal type D6 sites. If nothing crashes, I'll add it to two more production type sites in about a week.
Best,
Sam
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | blacklist_all_comment_spammers_option_1855304_1.patch | 7.3 KB | bryrock |
| httpbl.module.patch | 3.76 KB | Michael-IDA |
Comments
Comment #1
bryrock commentedChanged title of this issue, for clarity.
I tested the above patch on a pull of 6.x-2.x-dev. Worked for me.
Changed the new link added that indicated it went to an explanation of threat ratings. That was not where it was going.
Also did some more tidying of descriptions on the admin page.
Comment #2
Michael-IDA commentedHi Bryan,
I've installed the .dev and .patch on two non-production type sites. At first look all seems well (and fixing the typo's was a nice touch :).
I'll give them a week and check the log files and report back.
Best,
Sam
Edit:
Adding notes here so I don't lose them.
Site A: blocked 417 visits (401 blacklisted and 16 greylisted)
Site B: blocked 1027 visits (928 blacklisted and 99 greylisted)
Comment #3
Michael-IDA commentedTested on two sites, all seems well. I'd really like to test it on C: below, but it's D5 and never enough time in a day...
Still have problems getting valid data back from projecthoneypot.org, but that's on their end.
Notes:
A: blocked 505 visits (483 blacklisted and 22 greylisted)
B: blocked 1156 visits (1053 blacklisted and 103 greylisted)
C: blocked 7489 visits (7487 blacklisted and 2 greylisted).
Comment #4
bryrock commented