cloudflare / cf-ip-rewrite Goto Github PK

Typically having an empty function doing nothing is considered bad practise, in this case the constructor does not need to be in the class.

The load balancer has internal IP, so REMOTE_ADDR isn't from Cloudflare.

Something like CloudflareRemoteIPTrustedProxy in mod_cloudflare would be useful.

It would be great if the $cf_ipv4 and $cf_ipv6 where either public static variables, or at least have a static accessor to retrieve them.

Right now the only way to get my hand on the IP ranges without having IpRewrite change my global REMOTE_ADDR is this:

$inst = (new \ReflectionClass(\Cloudflare\IpRewrite::class))->newInstanceWithoutConstructor();
$ref = new ReflectionClass($inst);
$ipv4 = $ref->getProperty('cf_ipv4');
$ipv6 = $ref->getProperty('cf_ipv6');
$ipv4->setAccessible(true);
$ipv6->setAccessible(true);

var_dump($ipv4->getValue($inst));
var_dump($ipv6->getValue($inst));

Which is rather tedious.