ralxz

Hey @odoremieux, thanks for your patience! I just released v1.5 which contains this fix. Please give it a try and let me know if it worked!

Best,
Ryan

Thank you for the report! I was not aware of this ?? I will try to release a patch early next week.

Will report back here when released and would appreciate if you could take a look to see if it is fixed.

Thanks again,
Ryan

Hey there, thanks for the kind words!

Yes if you are comfortable commenting out some lines of code these two should do the trick:

Comment out this line:

https://github.com/ryanio/wp-limit-max-ips-per-user/blob/master/trunk/limit-max-ips-per-user.php#L433

and these lines:

https://github.com/ryanio/wp-limit-max-ips-per-user/blob/master/trunk/limit-max-ips-per-user.php#L457-L460

If you’d be interested in possibly funding some hours of work to accomplish this task, I should be able to do it in a few weeks when my schedule is more clear. Please let me know if that would be of interest to you. Thanks!

hi there, no there isn’t, that is a cool idea though, the code is open source if you’d be interested in adding something like that ?? unfortunately i don’t have the time right now.

Thanks for your message, let’s continue talking over email.

I have uploaded v1.1 of the plugin which should fix the issue, please let me know if you encounter any other problems ??

Hey metikar, thanks for your message.

I forgot about this plugin! I uploaded it 5 years ago and haven’t updated it since. I wasn’t sure if anyone was actually using it ??

It looks like WordPress updated their version of jQuery so I need to update the function in includes/pages/user-edit/user-edit.js from jQuery.live( to jQuery.on(. If you want to try it on your side and see if it fixes the issue, that would be great. I’ll see if I can get to updating the plugin sometime this weekend.

Great, glad to hear ??

Hey Mathijs,

I just looked into this, and it looks like if they are using IPV6 it should be logging correctly.

This is the code in the plugin:

    function get_user_ip() {
        $client = @$_SERVER['HTTP_CLIENT_IP'];
        $forward = @$_SERVER['HTTP_X_FORWARDED_FOR'];
        $remote = $_SERVER['REMOTE_ADDR'];

        if(filter_var($client, FILTER_VALIDATE_IP)) {
            $ip = $client;
        } elseif(filter_var($forward, FILTER_VALIDATE_IP)) {
            $ip = $forward;
        } else {
            $ip = $remote;
        }

        return $ip;
    }

If the server is accepting connections on an IPv6 socket, then it should report back correctly. If you are only getting IPv4 addresses for your users, maybe it’s something to due with your hosting configuration?

Let me know if I can be more help.

-Ryan

Hey Mathijs,

Thanks for the message. That’s a great idea, I’ll see if I can put in some logic to use their IPV6 address if available.

I’ll see if I can get to it this week.

Cheers,
Ryan

Thanks Nathan!

Yes the log automatically prunes entries older than however long the max limit is set. This keeps the database small and fast ??

Cheers,
Ryan