- π¬π§United Kingdom alexpott πͺπΊπ
-
+++ b/src/EventSubscriber/ClientIpRestore.php @@ -107,8 +109,9 @@ class ClientIpRestore implements EventSubscriberInterface { + if ($request_expected_to_bypass_cloudflare || in_array($client_ip, $client_ip_restore_whitelist)) {
in_array() should always have the 3rd param set to TRUE.
-
+++ b/src/Form/SettingsForm.php @@ -359,6 +373,7 @@ class SettingsForm extends FormBase implements ContainerInjectionInterface { + $client_ip_restore_whitelist = preg_split('/\r\n|\r|\n/', $form_state->getValue('client_ip_restore_whitelist'));
I'd consider trimming each element of the array and also validating the fact each element is an IP address.
-
- Status changed to Needs review
8 months ago 8:34am 27 March 2024 - last update
8 months ago Composer require failure - π©πͺGermany daniel.bosen
The patch for 2.0.0-alpha1 also adresses outstanding issues.
- last update
8 months ago Composer require failure