FIX Don't blindly assume that HOME is writable

bringyourownideas · Sep 13, 2018 · a2a3ad5 · a2a3ad5
1 parent 0618b7a
commit a2a3ad5
Showing 1 changed file with 6 additions and 2 deletions.
diff --git a/src/Extensions/ComposerLoaderExtension.php b/src/Extensions/ComposerLoaderExtension.php
@@ -100,8 +100,12 @@ protected function getInstalledConstraint(BaseRepository $repository, $packageNa
     public function onAfterBuild()
     {
         // Mock COMPOSER_HOME if it's not defined already. Composer requires one of the two to be set.
-        if (!getenv('HOME') && !getenv('COMPOSER_HOME')) {
-            putenv('COMPOSER_HOME=/tmp');
+        if (!getenv('COMPOSER_HOME')) {
+            $home = getenv('HOME');
+            if (!$home || !is_dir($home) || !is_writable($home)) {
+                // Set our own directory
+                putenv('COMPOSER_HOME=' . sys_get_temp_dir());
+            }
         }
 
         $originalDir = getcwd();