fix: file descriptor leaks in WriteApi (#95)

influxdata · Sep 29, 2021 · 8f52ccc · 8f52ccc
1 parent 3f18155
commit 8f52ccc
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,5 +1,8 @@
 ## 2.3.0 [unreleased]
 
+### Bug Fixes
+1. [#95](https://github.com/influxdata/influxdb-client-php/pull/95): Fix file descriptor leaks in WriteApi
+
 ## 2.2.0 [2021-09-17]
 
 ### Features

diff --git a/src/InfluxDB2/Client.php b/src/InfluxDB2/Client.php
@@ -118,6 +118,7 @@ public function close()
         foreach ($this->autoCloseable as $ac) {
             $ac->close();
         }
+        $this->autoCloseable = [];
     }
 
     public function getConfiguration()

diff --git a/src/InfluxDB2/WriteApi.php b/src/InfluxDB2/WriteApi.php
@@ -149,7 +149,11 @@ public function close()
     {
         $this->closed = true;
 
-        $this->worker()->flush();
+        if (isset($this->worker)) {
+            $this->worker()->flush();
+        }
+
+        unset($this->worker);
     }
 
     private function worker(): Worker