When configured to check requests for anonymous users "http:bl" will not check any IP address. The reason is that in line 631

if (variable_get('httpbl_check ... && $user )) {

global variable/struct "user" is always defined and the check results in TRUE.
Probably it was meant to check it the current user id is greater than 0:

... && $user-uid

See attached patch

CommentFileSizeAuthor
httpbl.module-user.patch468 bytesschildi
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

ruharen’s picture

ruharen CreditAttribution: ruharen commented
Title: No checks done when cofigured for anonymous users » No checks done when configured for anonymous users
Category: task » bug

Atachment is not working. Also, this is a bug.

praseodym’s picture

praseodym CreditAttribution: praseodym commented
Assigned: Unassigned » praseodym

I'll have to see if I can update the 5.x branch. I think a check for "$user->uid > 0" instead of just "$user" should suffice.

praseodym’s picture

praseodym CreditAttribution: praseodym commented
Status: Active » Fixed

A new release package should be ready in a few minutes.

Status: Fixed » Closed (fixed)

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