Replace localhost with private IP ranges fetcher

[aleho]

If #33 is merged, this needs a rebase.

[JasonLovesDoggo]

I do feel like the name private fetcher is a little lacking in detail but I couldn't think of anything better so that gets a pass for me.

[JasonLovesDoggo]

Would be nice if we could remove the overlapping IPS with localhost fetcher

[aleho]

I wasn't sure about that, but I'll change it.

[aleho]

I was trying to keep it close to Caddy, by the way: https://caddyserver.com/docs/caddyfile/matchers#remote-ip

[JasonLovesDoggo]

Actually, thinking about it for a minute. I feel like this could be instead layered on top of the localhost fetcher.

Perhaps rename that to local fetcher and add the missing CIDRs there

Sorry for the scattered replies, I'm just replying on my phone

[aleho]

Actually, thinking about it for a minute. I feel like this could be instead layered on top of the localhost fetcher.

Perhaps rename that to local fetcher and add the missing CIDRs there

The problem is, from a network perspective private IP ranges are not required to be local. 😁

[JasonLovesDoggo]

Actually, thinking about it for a minute. I feel like this could be instead layered on top of the localhost fetcher.

Perhaps rename that to local fetcher and add the missing CIDRs there

The problem is, from a network perspective private IP ranges are not required to be local. 😁

True! I really just created the localhost fetcher for development of this plugin. Not much thought was put into it.

You're welcome to update the PR so this new module is a direct replacement for the localhost fetcher

[JasonLovesDoggo]

LGTM! Thanks for the contributions!