IP Blocker for v1.5.x

Quote:

---

Originally Posted by picandnix

That sounds perfect to me :hug:

---

That's the tack I'll take then; I'll update here when it's ready.

Quote:

---

Originally Posted by lat9

That's the tack I'll take then; I'll update here when it's ready.

---

You star!

Thank You!

Quote:

---

Originally Posted by lat9

I've submitted v2.0.1 to the plugins area to correct the issue discovered by kitcorsa and to add a blocked-IP, failed address lockout count to the plugin's configuration.

---

Now available via the plugins ...

Was just looking over the code... (Trying to see if can incorporate into another plug-in if both are installed.)

When looking at init_ip_blocker.php of the includes/init_includes directory... Seems like (and maybe I'm wrong) that upon completion of the file, the session 'ip_blocker_pass' variable is always false... Even if momentarily previously set as true inside the loop. Do I misunderstand what the code does? Ie. is there extra code that doesn't really do accomplish anything?

I'll grant that it looks funny; is it not working?

Quote:

---

Originally Posted by lat9

I'll grant that it looks funny; is it not working?

---

So far no complaints, it was that I was taking a look to see what I would need to incorporate to add a link to choose to block an address, and so had to follow the code through. Wasn't exactly digging for each line, but the general flow. That set of instructions (setting to true in a loop/if statement) made me wonder if there was something listening for a change in the setting. If there was, then I wanted to follow through with what that code did and wanted to find it. The call to change it doesn't use the notifier code, but having not looked at everything, thought I would ask the developer. :)

Are you considering implementing a sort of consolidation function into this in the future?

Ie. Inspect all entries, if two or more are consecutive to change from two independent entries, to a single combined? And if perchance all in a subset have been added, to change to that subset? (Thus preventing the overlap described in the readme.)

Quote:

---

Originally Posted by mc12345678

Are you considering implementing a sort of consolidation function into this in the future?

Ie. Inspect all entries, if two or more are consecutive to change from two independent entries, to a single combined? And if perchance all in a subset have been added, to change to that subset? (Thus preventing the overlap described in the readme.)

---

Nope, no plans to do this.

Quote:

---

Originally Posted by mc12345678

So far no complaints, it was that I was taking a look to see what I would need to incorporate to add a link to choose to block an address, and so had to follow the code through. Wasn't exactly digging for each line, but the general flow. That set of instructions (setting to true in a loop/if statement) made me wonder if there was something listening for a change in the setting. If there was, then I wanted to follow through with what that code did and wanted to find it. The call to change it doesn't use the notifier code, but having not looked at everything, thought I would ask the developer. :)

---

I've submitted v2.0.2 to the plugins area to correct the session-based variable handling and to remove the debug code display from the "special login" page.