feat(*): Prepare release 4.3.0

Author: julienloizelet

CHANGELOG.md

@@ -15,8 +15,8 @@ As far as possible, we try to adhere to [Symfony guidelines](https://symfony.com
 
 ---
 
-## [4.3.0](https://github.com/crowdsecurity/php-remediation-engine/releases/tag/v4.3.0) - 2025-04-??
-[_Compare with previous release_](https://github.com/crowdsecurity/php-remediation-engine/compare/v4.2.0...HEAD)
+## [4.3.0](https://github.com/crowdsecurity/php-remediation-engine/releases/tag/v4.3.0) - 2025-04-11
+[_Compare with previous release_](https://github.com/crowdsecurity/php-remediation-engine/compare/v4.2.0...v4.3.0)
 
 
 ### Added

src/AbstractRemediation.php

@@ -561,7 +561,7 @@ private function validateRawDecision(array $rawDecision): bool
             && !empty($rawDecision['duration'])
         ) {
             $result = true;
-            // We don't want blocklists decisions without a scenario
+            // We don't want blocklists or allowlists decisions without a scenario
             if (
                 Constants::ORIGIN_LISTS === $rawDecision['origin']
                 && empty($rawDecision['scenario'])

src/CapiRemediation.php

@@ -207,7 +207,7 @@ private function handleAllowListDecisions(array $allowlists): array
                 }
             }
         } catch (\Exception $e) {
-            $this->logger->info('Something went wrong during list decisions process', [
+            $this->logger->info('Something went wrong during allowlists decisions process', [
                 'type' => 'CAPI_REM_HANDLE_ALLOW_LIST_DECISIONS',
                 'message' => $e->getMessage(),
                 'code' => $e->getCode(),
@@ -232,15 +232,19 @@ private function handleAllowListResponse(string $listResponse, array $allowDecis
     {
         $decisions = [];
         $listedAllows = explode(\PHP_EOL, $listResponse);
-        $this->logger->debug('Handle allow list decisions', [
+        $this->logger->debug('Handle allowlists decisions', [
             'type' => 'CAPI_REM_HANDLE_ALLOW_LIST_DECISIONS',
             'list_count' => count($listedAllows),
         ]);
         foreach ($listedAllows as $listedAllow) {
             $decoded = json_decode($listedAllow, true);
             $allowDecision['value'] = $decoded['value'];
             $allowDecision['scope'] = $decoded['scope'];
-            $allowDecision['duration'] = '1s'; // Will be overwritten by the duration in the allowlist
+            /*
+             * This hardcoded value ill be overwritten below by the duration in the allowlist.
+             * We have to set it to avoid an exception from the convertRawDecision method.
+             */
+            $allowDecision['duration'] = '1s';
             $decision = $this->convertRawDecision($allowDecision);
 
             if ($decision) {
@@ -317,7 +321,7 @@ private function handleListDecisions(array $blocklists): array
                 }
             }
         } catch (\Exception $e) {
-            $this->logger->info('Something went wrong during list decisions process', [
+            $this->logger->info('Something went wrong during blocklists decisions process', [
                 'type' => 'CAPI_REM_HANDLE_LIST_DECISIONS',
                 'message' => $e->getMessage(),
                 'code' => $e->getCode(),
@@ -351,7 +355,7 @@ private function handleListResponse(string $listResponse, array $blockDecision):
     {
         $decisions = [];
         $listedIps = explode(\PHP_EOL, $listResponse);
-        $this->logger->debug('Handle list decisions', [
+        $this->logger->debug('Handle blocklist decisions', [
             'type' => 'CAPI_REM_HANDLE_LIST_DECISIONS',
             'list_count' => count($listedIps),
         ]);

src/Constants.php

@@ -98,5 +98,5 @@ class Constants extends CommonConstants
     /**
      * @var string The current version of this library
      */
-    public const VERSION = 'v4.2.0';
+    public const VERSION = 'v4.3.0';
 }