From 25fabcc1be859e66cfd4b4785809f87f7e238cb2 Mon Sep 17 00:00:00 2001 From: Jeremy Price Date: Mon, 8 Jul 2024 22:12:09 -0700 Subject: [PATCH 1/2] More helpful output when we can't read .upgrade_requirements.json Handle json decode errors and display the raw output in such cases, to better understand what's happening and help debugging --- upgrade.php | 12 ++++++++++-- 1 file changed, 10 insertions(+), 2 deletions(-) diff --git a/upgrade.php b/upgrade.php index 0215c3937..6c4146b3d 100644 --- a/upgrade.php +++ b/upgrade.php @@ -59,8 +59,8 @@ echo "- clear out old cache settings\n\n"; // Fetching most current upgrade requirements from github. Read more here: https://github.com/snipe/snipe-it/pull/14127 $remote_requirements_file = "https://raw.githubusercontent.com/snipe/snipe-it/$branch/.upgrade_requirements.json"; -$upgrade_requirements = json_decode(url_get_contents($remote_requirements_file), true); - +$upgrade_requirements_raw = url_get_contents($remote_requirements_file); +$upgrade_requirements = json_decode($upgrade_requirements_raw, true); if (! $upgrade_requirements) { if(!$skip_php_checks){ echo "\nERROR: Failed to retrieve remote requirements from $remote_requirements_file\n\n"; @@ -68,6 +68,14 @@ if (! $upgrade_requirements) { echo "NOTE: You passed a custom branch: $branch\n"; echo " If the above URL doesn't work, that may be why. Please check you branch spelling/extistance\n\n"; } + + if (json_last_error()) { + print "JSON DECODE ERROR DETECTED:\n"; + print json_last_error_msg() . "\n\n"; + print "Raw curl output:\n"; + print $upgrade_requirements_raw . "\n\n"; + } + echo "We suggest correcting this, but if you can't, please verify that your requirements conform to those at that url.\n\n"; echo " -- DANGER -- DO AT YOUR OWN RISK --\n"; echo " IF YOU ARE SURE, re-run this script with --skip-php-compatibility-checks to skip this check.\n"; From ec063b4af50a8181189c08f7558395b9863a3cc4 Mon Sep 17 00:00:00 2001 From: snipe Date: Wed, 10 Jul 2024 17:07:44 +0100 Subject: [PATCH 2/2] Updated PHP min version Signed-off-by: snipe --- config/app.php | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/config/app.php b/config/app.php index 3ab0d632b..61c06177f 100755 --- a/config/app.php +++ b/config/app.php @@ -242,7 +242,7 @@ return [ | */ - 'min_php' => '7.4.0', + 'min_php' => '8.1.2', /*