The package doesn't work with Symfony 3.3.0+

[mlocati]

This package calls directly Request::setTrustedProxies(), which requires a 2nd parameter since Symfony 3.3.0+.

concretecms/concretecms#6711 fixes the Request::setTrustedProxies() call for the core, so IMHO the package should simply store its data to concrete.security.trusted_proxies.ips and let the core handle it.

If it's ok, I have some code that already implements it, and I can create a PR.

[aembler]

I agree, let's just make this package one that retrieves the IPs, and use the core mechanisms to set them.

[KorvinSzanto]

What @aembler describes is already what this package is. I think if we'd do this in a different way, we'd use $config->set(...) to overwrite the core settings at runtime similar to how we are doing it now.