Github CI/CD doesn't allow set_time_limit so cope with that

PennyDreadfulMTG · Oct 31, 2024 · 7e844ac · 7e844ac
1 parent 447eff6
commit 7e844ac
Show file tree

Hide file tree

Showing 3 changed files with 23 additions and 6 deletions.
diff --git a/gatherling/admin/db-upgrade.php b/gatherling/admin/db-upgrade.php
@@ -6,12 +6,18 @@
 
 use Gatherling\Data\Setup;
 
+use function Gatherling\Helpers\logger;
 use function Gatherling\Helpers\server;
 use function Safe\set_time_limit;
 
 require_once __DIR__ . '/../lib.php';
 
-set_time_limit(0);
+try {
+    set_time_limit(0);
+} catch (\Exception $e) {
+    // set_time_limit is not allowed here but we'll try our best to complete in time.
+    logger()->warning('Failed to set time limit, trying anyway: ' . $e->getMessage());
+}
 
 function main(): never
 {

diff --git a/gatherling/insertcardset.php b/gatherling/insertcardset.php
@@ -8,15 +8,21 @@
 use Gatherling\Views\Redirect;
 
 use function Gatherling\Helpers\files;
+use function Gatherling\Helpers\logger;
 use function Gatherling\Helpers\request;
 use function Gatherling\Helpers\server;
 use function Safe\set_time_limit;
 
-set_time_limit(0);
-mysqli_report(MYSQLI_REPORT_ERROR | MYSQLI_REPORT_STRICT);
-
 require_once __DIR__ . '/lib.php';
 
+try {
+    set_time_limit(0);
+} catch (\Exception $e) {
+    // set_time_limit is not allowed here but we'll try our best to complete in time.
+    logger()->warning('Failed to set time limit, running anyway: ' . $e->getMessage());
+}
+mysqli_report(MYSQLI_REPORT_ERROR | MYSQLI_REPORT_STRICT);
+
 function main(): never
 {
     global $argv;

diff --git a/gatherling/util/updateDefaultFormats.php b/gatherling/util/updateDefaultFormats.php
@@ -14,10 +14,15 @@
 use function Gatherling\Helpers\server;
 use function Safe\set_time_limit;
 
-set_time_limit(0);
-
 require_once __DIR__ . '/../lib.php';
 
+try {
+    set_time_limit(0);
+} catch (\Exception $e) {
+    // set_time_limit is not allowed here but we'll try our best to complete in time.
+    logger()->warning('Failed to set time limit, running anyway: ' . $e->getMessage());
+}
+
 function main(): never
 {
     if (PHP_SAPI != 'cli' && $_SERVER['REQUEST_METHOD'] == 'GET') { // unauthorized POST is okay