Skip to content

Commit 69065bb

Browse files
fix: correctly interpret ini values in PhpSettingsChecker (#303)
1 parent 8b0e7f0 commit 69065bb

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

src/Components/Health/Checker/PerformanceChecker/PhpSettingsChecker.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,7 @@ private function checkAssertActive(HealthCollection $collection, string $url): v
3939
private function checkEnableFileOverride(HealthCollection $collection, string $url): void
4040
{
4141
$currentValue = $this->iniGetFailover('opcache.enable_file_override');
42-
if ($currentValue !== '1') {
42+
if (!\filter_var($currentValue, \FILTER_VALIDATE_BOOL)) {
4343
$collection->add(
4444
SettingsResult::warning(
4545
'php.opcache.enable_file_override',
@@ -71,7 +71,7 @@ private function checkInternedStringsBuffer(HealthCollection $collection, string
7171
private function checkZendDetectUnicode(HealthCollection $collection, string $url): void
7272
{
7373
$currentValue = $this->iniGetFailover('zend.detect_unicode');
74-
if ($currentValue !== '0') {
74+
if (\filter_var($currentValue, \FILTER_VALIDATE_BOOL)) {
7575
$collection->add(
7676
SettingsResult::warning(
7777
'php.zend.detect_unicode',

0 commit comments

Comments
 (0)