Merge branch '1.0' into 1

README.md

@@ -38,6 +38,19 @@ composer require bringyourownideas/silverstripe-composer-update-checker ^1
 php ./framework/cli-script.php dev/build flush=1
 php ./framework/cli-script.php dev/tasks/UpdatePackageInfoTask
 ```
+
+## Note for private repositories
+
+Please note that if your project has modules that are stored in private repositories, the server running the BuildTask
+will need to have the necessary permissions to access the private VCS repositories in order for the report to include
+update information about necessary updates to the module.
+
+If the process looking for available updates fails (for example, due to an authentication failure against a private
+repository) the process will fail gracefully and allow the rest of the report generation to continue.
+
+Users on the [Common Web Platform](https://cwp.govt.nz) will currently not be able to retrieve information about
+updates to private repositories.
+
 ## Documentation
 
 Please see the user guide section of the [SilverStripe Maintenance module](https://github.com/bringyourownideas/silverstripe-maintenance/tree/1/docs/en/userguide).

src/Extensions/CheckComposerUpdatesExtension.php

@@ -5,6 +5,8 @@
 use BringYourOwnIdeas\UpdateChecker\UpdateChecker;
 use Config;
 use Extension;
+use RuntimeException;
+use SS_Log;
 use UpdatePackageInfoTask;
 
 /**
@@ -51,9 +53,18 @@ public function updatePackageInfo(array &$installedPackageList)
             }
             $packageData = $composerPackagesAndConstraints[$packageName];
 
-            // Check for a relevant update version to recommend returned as keyed array and add to existing package
-            // details array
-            $updates = $this->getUpdateChecker()->checkForUpdates($packageData['package'], $packageData['constraint']);
+            try {
+                // Check for a relevant update version to recommend returned as keyed array and add to existing package
+                // details array
+                $updates = $this->getUpdateChecker()
+                    ->checkForUpdates($packageData['package'], $packageData['constraint']);
+            } catch (RuntimeException $ex) {
+                // If exceptions are thrown during execution, fail gracefully and allow the rest of the report
+                // generation to continue
+                $updates = [];
+                SS_Log::log($ex->getMessage(), SS_Log::WARN);
+            }
+
             $installedPackage = array_merge($installedPackage, $updates);
         }
     }