Fix process resource detector triggers warning if running with unknow…

…n uid (#840)
open-telemetry · Oct 9, 2022 · 7940773 · 7940773
1 parent 8562d0d
commit 7940773
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/src/SDK/Resource/Detectors/Process.php b/src/SDK/Resource/Detectors/Process.php
@@ -27,8 +27,9 @@ public function getResource(): ResourceInfo
             $attributes[ResourceAttributes::PROCESS_COMMAND_ARGS] = $_SERVER['argv'];
         }
 
-        if (extension_loaded('posix')) {
-            $attributes[ResourceAttributes::PROCESS_OWNER] = \posix_getpwuid(\posix_geteuid())['name'];
+        /** @phan-suppress-next-line PhanTypeComparisonFromArray */
+        if (extension_loaded('posix') && ($user = \posix_getpwuid(\posix_geteuid())) !== false) {
+            $attributes[ResourceAttributes::PROCESS_OWNER] = $user['name'];
         }
 
         return ResourceInfo::create(Attributes::create($attributes), ResourceAttributes::SCHEMA_URL);