Merge pull request #56506 from nextcloud/backport/56438/stable32

[stable32] feat(profiler): Use gzdecode/gzencode to parse profiles
pull/56760/head
Joas Schilling 2025-11-30 21:36:44 +07:00 committed by GitHub
commit c2083c990c
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
1 changed files with 41 additions and 11 deletions

@ -17,6 +17,17 @@ class FileProfilerStorage {
// Folder where profiler data are stored.
private string $folder;
/** @psalm-suppress UndefinedClass */
public const allowedClasses = [
\OCA\Profiler\DataCollector\EventLoggerDataProvider::class,
\OCA\Profiler\DataCollector\HttpDataCollector::class,
\OCA\Profiler\DataCollector\MemoryDataCollector::class,
\OCA\User_LDAP\DataCollector\LdapDataCollector::class,
\OC\Memcache\ProfilerWrapperCache::class,
\OC\Profiler\RoutingDataCollector::class,
\OC\DB\DbDataCollector::class,
];
/**
* Constructs the file storage using a "dsn-like" path.
*
@ -97,11 +108,21 @@ class FileProfilerStorage {
return null;
}
if (\function_exists('gzcompress')) {
$file = 'compress.zlib://' . $file;
$h = fopen($file, 'r');
flock($h, \LOCK_SH);
$data = stream_get_contents($h);
flock($h, \LOCK_UN);
fclose($h);
if (\function_exists('gzdecode')) {
$data = @gzdecode($data) ?: $data;
}
if (!$data = unserialize($data, ['allowed_classes' => self::allowedClasses])) {
return null;
}
return $this->createProfileFromData($token, unserialize(file_get_contents($file)));
return $this->createProfileFromData($token, $data);
}
/**
@ -139,14 +160,13 @@ class FileProfilerStorage {
'status_code' => $profile->getStatusCode(),
];
$context = stream_context_create();
$data = serialize($data);
if (\function_exists('gzcompress')) {
$file = 'compress.zlib://' . $file;
stream_context_set_option($context, 'zlib', 'level', 3);
if (\function_exists('gzencode')) {
$data = gzencode($data, 3);
}
if (file_put_contents($file, serialize($data), 0, $context) === false) {
if (file_put_contents($file, $data, \LOCK_EX) === false) {
return false;
}
@ -266,11 +286,21 @@ class FileProfilerStorage {
continue;
}
if (\function_exists('gzcompress')) {
$file = 'compress.zlib://' . $file;
$h = fopen($file, 'r');
flock($h, \LOCK_SH);
$data = stream_get_contents($h);
flock($h, \LOCK_UN);
fclose($h);
if (\function_exists('gzdecode')) {
$data = @gzdecode($data) ?: $data;
}
if (!$data = unserialize($data, ['allowed_classes' => self::allowedClasses])) {
continue;
}
$profile->addChild($this->createProfileFromData($token, unserialize(file_get_contents($file)), $profile));
$profile->addChild($this->createProfileFromData($token, $data, $profile));
}
return $profile;