Merge pull request #26747 from nextcloud/php8-libxml
Fix installer deprecation warnings for PHP 8
This commit is contained in:
commit
e1a3000cbe
|
@ -307,7 +307,10 @@ class Installer {
|
||||||
// Check if the signature actually matches the downloaded content
|
// Check if the signature actually matches the downloaded content
|
||||||
$certificate = openssl_get_publickey($app['certificate']);
|
$certificate = openssl_get_publickey($app['certificate']);
|
||||||
$verified = (bool)openssl_verify(file_get_contents($tempFile), base64_decode($app['releases'][0]['signature']), $certificate, OPENSSL_ALGO_SHA512);
|
$verified = (bool)openssl_verify(file_get_contents($tempFile), base64_decode($app['releases'][0]['signature']), $certificate, OPENSSL_ALGO_SHA512);
|
||||||
openssl_free_key($certificate);
|
// PHP 8+ deprecates openssl_free_key and automatically destroys the key instance when it goes out of scope
|
||||||
|
if ((PHP_VERSION_ID < 80000)) {
|
||||||
|
openssl_free_key($certificate);
|
||||||
|
}
|
||||||
|
|
||||||
if ($verified === true) {
|
if ($verified === true) {
|
||||||
// Seems to match, let's proceed
|
// Seems to match, let's proceed
|
||||||
|
@ -339,9 +342,13 @@ class Installer {
|
||||||
}
|
}
|
||||||
|
|
||||||
// Check if appinfo/info.xml has the same app ID as well
|
// Check if appinfo/info.xml has the same app ID as well
|
||||||
$loadEntities = libxml_disable_entity_loader(false);
|
if ((PHP_VERSION_ID < 80000)) {
|
||||||
$xml = simplexml_load_file($extractDir . '/' . $folders[0] . '/appinfo/info.xml');
|
$loadEntities = libxml_disable_entity_loader(false);
|
||||||
libxml_disable_entity_loader($loadEntities);
|
$xml = simplexml_load_file($extractDir . '/' . $folders[0] . '/appinfo/info.xml');
|
||||||
|
libxml_disable_entity_loader($loadEntities);
|
||||||
|
} else {
|
||||||
|
$xml = simplexml_load_file($extractDir . '/' . $folders[0] . '/appinfo/info.xml');
|
||||||
|
}
|
||||||
if ((string)$xml->id !== $appId) {
|
if ((string)$xml->id !== $appId) {
|
||||||
throw new \Exception(
|
throw new \Exception(
|
||||||
sprintf(
|
sprintf(
|
||||||
|
|
Loading…
Reference in New Issue