Posix & Pcntl are now optionnal but can be enabled with doPosixCheck config.

Author: Geolim4

README.md

@@ -1,21 +1,4 @@
-## Support & Security
-
-Support for this extension must be posted to the main [Phpfastcache repository](https://github.com/PHPSocialNetwork/phpfastcache/issues).
-
-## Composer installation:
-
-```php
-composer install phpfastcache/couchbasev4-extension
-```
-
-This extension requires:
-
-- The PHP `Couchbase` extension 4.x at least
-- The composer `Couchbase/Couchbase` library 4.x at least
-- The PHP `Posix` to fix a known bug [PCBC-886](https://issues.couchbase.com/projects/PCBC/issues/PCBC-886)
-
-
-## Contributing
+## Contributing [![PHP Tests](https://github.com/PHPSocialNetwork/couchbasev4-extension/actions/workflows/php.yml/badge.svg)](https://github.com/PHPSocialNetwork/couchbasev4-extension/actions/workflows/php.yml)
 Merge requests are welcome but will require the tests plus the quality tools to pass:
 
 _(Commands must be run from the repository root)_
@@ -37,3 +20,32 @@ _(Commands must be run from the repository root)_
 ```bash
 ./vendor/bin/phpstan analyse lib/ -l 6 -c phpstan.neon 2>&1
 ```
+
+## Support & Security
+
+Support for this extension must be posted to the main [Phpfastcache repository](https://github.com/PHPSocialNetwork/phpfastcache/issues).
+
+## Composer installation:
+
+```php
+composer install phpfastcache/couchbasev4-extension
+```
+
+#### ⚠️ This extension requires:
+
+1️⃣ The PHP `Couchbase` extension 4.x at least
+
+2️⃣ The composer `Couchbase/Couchbase` library 4.x at least
+
+#### ⚠️ This extension optionally requires: 
+1️⃣ The PHP `Posix` to fix a known Couchbase Extension bug [PCBC-886](https://issues.couchbase.com/projects/PCBC/issues/PCBC-886).  
+Once this bug has been fixed the dependency suggestion will be removed. 
+If your application do manipulate processes with `Posix` extension, and you want the fix to be enabled, set up the config like this:
+```php
+$config = (new CouchbaseConfig())->setDoPosixCheck(true);
+```
+
+2️⃣ Also the PHP `Pcntl` if you plan to contribute to this project and run the tests before pushing your Merge Request.
+
+
+

composer.json

@@ -24,18 +24,19 @@
         "php": ">=8.0",
         "phpfastcache/phpfastcache": "^9.2",
         "couchbase/couchbase": "^4.0",
-        "ext-posix": "*",
         "ext-couchbase": "^4.0"
     },
+    "suggest": {
+        "ext-posix": "*",
+        "ext-pcntl": "*"
+    },
     "require-dev": {
         "webmozart/assert": "^1.11",
-        "ext-pcntl": "*",
         "phpmd/phpmd": "@stable",
         "squizlabs/php_codesniffer": "@stable",
         "phpstan/phpstan": "^1.5",
         "jetbrains/phpstorm-stubs": "dev-master"
     },
-    "suggest": {},
     "autoload": {
         "psr-4": {
             "Phpfastcache\\Extensions\\": "lib/Phpfastcache/Extensions"

lib/Phpfastcache/Extensions/Drivers/Couchbasev4/Config.php

@@ -19,6 +19,7 @@
 
 use Couchbase\ClusterOptions;
 use Phpfastcache\Config\ConfigurationOption;
+use Phpfastcache\Exceptions\PhpfastcacheInvalidArgumentException;
 use Phpfastcache\Exceptions\PhpfastcacheLogicException;
 
 class Config extends ConfigurationOption
@@ -38,6 +39,7 @@ class Config extends ConfigurationOption
     protected bool $secure            = false;
     protected bool $allowFlush        = true;
     protected bool $flushFailSilently = false;
+    protected bool $doPosixCheck      = false;
 
     protected ?ClusterOptions $clusterOptions = null;
 
@@ -222,4 +224,18 @@ public function getClusterOptions(): ClusterOptions
         }
         return $this->clusterOptions;
     }
+
+    public function isDoPosixCheck(): bool
+    {
+        return $this->doPosixCheck;
+    }
+
+    public function setDoPosixCheck(bool $doPosixCheck): static
+    {
+        if ($doPosixCheck && !extension_loaded('posix')) {
+            throw new PhpfastcacheInvalidArgumentException('Posix extension is required to enable the doPosixCheck config entry.');
+        }
+
+        return $this->setProperty('doPosixCheck', $doPosixCheck);
+    }
 }

lib/Phpfastcache/Extensions/Drivers/Couchbasev4/Driver.php

@@ -69,12 +69,12 @@ public function __construct(ConfigurationOption $config, string $instanceId, Eve
      */
     public function driverCheck(): bool
     {
-        return extension_loaded('couchbase') && extension_loaded('posix');
+        return extension_loaded('couchbase') && (!$this->getConfig()->isDoPosixCheck() || extension_loaded('posix'));
     }
 
     public function getHelp(): string
     {
-        return 'Couchbasev4 requires the `php-couchbase` extension 4.x and the `php-posix` extension';
+        return 'Couchbasev4 requires the `php-couchbase` extension 4.x and optionally the `php-posix` extension if you enabled the config "doPosixCheck".';
     }
 
     /**
@@ -92,7 +92,9 @@ protected function driverConnect(): bool
             throw new PhpfastcacheDriverCheckException("You are using Couchbase extension $extVersion, You need to use a Couchbase V4 extension");
         }
 
-        $this->currentParentPID = posix_getppid();
+        if ($this->getConfig()->isDoPosixCheck() && extension_loaded('posix')) {
+            $this->currentParentPID = posix_getppid();
+        }
 
         $schema = $this->getConfig()->getSecure() ? 'couchbases' : 'couchbase';
         $servers = $this->getConfig()->getServers();
@@ -118,8 +120,10 @@ protected function driverConnect(): bool
      */
     protected function checkCurrentParentPID(): void
     {
-        if ($this->currentParentPID !== posix_getppid()) {
-            $this->driverConnect();
+        if ($this->getConfig()->isDoPosixCheck() && extension_loaded('posix')) {
+            if ($this->currentParentPID !== posix_getppid()) {
+                $this->driverConnect();
+            }
         }
     }
 

tests/Couchbasev4.test.php

@@ -26,6 +26,7 @@
 }
 
 $config = (new CouchbaseConfig(include $configFileName))
+    ->setDoPosixCheck(true)
     ->setUseStaticItemCaching(false);
 
 $cacheInstance = CacheManager::getInstance('Couchbasev4', $config);