Looking in http://api.drupal.org/api/drupal/modules%21user%21user.module/function/u... we can see that there a 2 kind of validations on user:

  1. Pure UID: Drupal validate only against user. This is the most strict.
  2. UID+IP: Drupal checks by this pair. This is used as it is safe enough but more practical because avoids locking out public users.

Drupal defaults to the second one. The purpose of this module is to unlock flood variables and let admins configure them.

Add also user_failed_login_identifier_uid_only to form.



abautu’s picture

Status: Active » Needs review
1020 bytes

Attaching a patch for this feature.

claudiu.cristea’s picture

Status: Needs review » Reviewed & tested by the community


claudiu.cristea’s picture

Issue summary: View changes


crystaldawn’s picture

Issue summary: View changes

I think this is probably out of scope for this particular project, but it might make sense as a feature request for cbp (which uses this module here as a dependency for the flood settings themselves).

heddn’s picture

1.25 KB

RTBC from me. It makes sense to expose the ability to configure the flood logging identifier in flood_control. However, attaching a re-roll to make things go easier. The current patch didn't want to apply cleanly.

joelpittet’s picture

RTBC++ thanks for fixing the patch. The trick looks like #1 forgot --relative flag when creating the patch inside their drupal site.