check if temp is writable

stefanak-michal · stefanak-michal · commit 3a79ee262890 · 2024-07-15T19:40:06.000+02:00
diff --git a/src/Bolt.php b/src/Bolt.php
@@ -31,7 +31,7 @@ public function __construct(private IConnection $connection)
             mkdir($_ENV['TEMP_DIR'], recursive: true);
         }
 
-        if (!getenv('BOLT_ANALYTICS_OPTOUT')) {
+        if (!getenv('BOLT_ANALYTICS_OPTOUT') && is_writable($_ENV['TEMP_DIR'] . DIRECTORY_SEPARATOR)) {
             if (!file_exists($_ENV['TEMP_DIR'] . DIRECTORY_SEPARATOR . 'php-bolt-analytics' . DIRECTORY_SEPARATOR)) {
                 mkdir($_ENV['TEMP_DIR'] . DIRECTORY_SEPARATOR . 'php-bolt-analytics');
             }
diff --git a/src/protocol/AProtocol.php b/src/protocol/AProtocol.php
@@ -73,9 +73,11 @@ protected function write(iterable $generator): void
 
     private function track(): void
     {
-        $file = $_ENV['TEMP_DIR'] . DIRECTORY_SEPARATOR . 'php-bolt-analytics' . DIRECTORY_SEPARATOR . 'queries.' . strtotime('today') . '.cnt';
-        $count = file_exists($file) ? intval(file_get_contents($file)) : 0;
-        file_put_contents($file, $count + 1);
+        if (is_writable($_ENV['TEMP_DIR']. DIRECTORY_SEPARATOR)) {
+            $file = $_ENV['TEMP_DIR'] . DIRECTORY_SEPARATOR . 'php-bolt-analytics' . DIRECTORY_SEPARATOR . 'queries.' . strtotime('today') . '.cnt';
+            $count = file_exists($file) ? intval(file_get_contents($file)) : 0;
+            file_put_contents($file, $count + 1);
+        }
     }
 
     /**
diff --git a/tests/protocol/ProtocolLayer.php b/tests/protocol/ProtocolLayer.php
@@ -105,7 +105,7 @@ protected function setUp(): void
             mkdir($_ENV['TEMP_DIR'], recursive: true);
         }
 
-        if (!getenv('BOLT_ANALYTICS_OPTOUT')) {
+        if (!getenv('BOLT_ANALYTICS_OPTOUT') && is_writable($_ENV['TEMP_DIR'] . DIRECTORY_SEPARATOR)) {
             if (!file_exists($_ENV['TEMP_DIR'] . DIRECTORY_SEPARATOR . 'php-bolt-analytics' . DIRECTORY_SEPARATOR)) {
                 mkdir($_ENV['TEMP_DIR'] . DIRECTORY_SEPARATOR . 'php-bolt-analytics');
             }

Original file line number	Diff line number	Diff line change
`@@ -31,7 +31,7 @@ public function __construct(private IConnection $connection)`
`31`	`31`	`mkdir($_ENV['TEMP_DIR'], recursive: true);`
`32`	`32`	`}`
`33`	`33`
`34`		`- if (!getenv('BOLT_ANALYTICS_OPTOUT')) {`
	`34`	`+ if (!getenv('BOLT_ANALYTICS_OPTOUT') && is_writable($_ENV['TEMP_DIR'] . DIRECTORY_SEPARATOR)) {`
`35`	`35`	`if (!file_exists($_ENV['TEMP_DIR'] . DIRECTORY_SEPARATOR . 'php-bolt-analytics' . DIRECTORY_SEPARATOR)) {`
`36`	`36`	`mkdir($_ENV['TEMP_DIR'] . DIRECTORY_SEPARATOR . 'php-bolt-analytics');`
`37`	`37`	`}`
Original file line number	Diff line number	Diff line change
`@@ -73,9 +73,11 @@ protected function write(iterable $generator): void`
`73`	`73`
`74`	`74`	`private function track(): void`
`75`	`75`	`{`
`76`		`- $file = $_ENV['TEMP_DIR'] . DIRECTORY_SEPARATOR . 'php-bolt-analytics' . DIRECTORY_SEPARATOR . 'queries.' . strtotime('today') . '.cnt';`
`77`		`- $count = file_exists($file) ? intval(file_get_contents($file)) : 0;`
`78`		`- file_put_contents($file, $count + 1);`
	`76`	`+ if (is_writable($_ENV['TEMP_DIR']. DIRECTORY_SEPARATOR)) {`
	`77`	`+ $file = $_ENV['TEMP_DIR'] . DIRECTORY_SEPARATOR . 'php-bolt-analytics' . DIRECTORY_SEPARATOR . 'queries.' . strtotime('today') . '.cnt';`
	`78`	`+ $count = file_exists($file) ? intval(file_get_contents($file)) : 0;`
	`79`	`+ file_put_contents($file, $count + 1);`
	`80`	`+ }`
`79`	`81`	`}`
`80`	`82`
`81`	`83`	`/**`
Original file line number	Diff line number	Diff line change
`@@ -105,7 +105,7 @@ protected function setUp(): void`
`105`	`105`	`mkdir($_ENV['TEMP_DIR'], recursive: true);`
`106`	`106`	`}`
`107`	`107`
`108`		`- if (!getenv('BOLT_ANALYTICS_OPTOUT')) {`
	`108`	`+ if (!getenv('BOLT_ANALYTICS_OPTOUT') && is_writable($_ENV['TEMP_DIR'] . DIRECTORY_SEPARATOR)) {`
`109`	`109`	`if (!file_exists($_ENV['TEMP_DIR'] . DIRECTORY_SEPARATOR . 'php-bolt-analytics' . DIRECTORY_SEPARATOR)) {`
`110`	`110`	`mkdir($_ENV['TEMP_DIR'] . DIRECTORY_SEPARATOR . 'php-bolt-analytics');`
`111`	`111`	`}`