Question about _httpbl_cache_update()

Issue Summary

In this function the code for adding a blacklisted IP to {access}, thereby banning it, is inoperable by virtue of being commented out. Is this an oversight or is there a reason for this?

As the module works now, overall, any IP blacklisted after a DNS lookup at Honeypot will be added to {access} and be banned, but if a greylisted IP attempts and fails the whitelisting challenge, it will be blacklisted only in {httpbl} but, because the code is commented out, not be banned altogether.

I meant to bring this up a few days ago, but I've been using the module on four sites with this code being operable, so I forgot about it being "officially" commented out until I reviewed the latest snapshot. It seems to work great, by the way.

Also, immediately above the commented-out code is a ToDo note in comments about checking for duplicates, but it seems that would not be necessary since a banned site would not be capable of gaining access for future blacklisting consideration until after the first ban has expired.