icinga2-diagnostics icon indicating copy to clipboard operation
icinga2-diagnostics copied to clipboard

Bug at Line 588?

Open wolf-in-sheeps-clothing opened this issue 5 years ago • 1 comments

Hi,

the script found 2 php.ini. All of them have a correct date.timezone. But in the output I see this:

  • At least one php.ini file has no valid timezone setting

At Line 27 you set PHPINITIMEZONEMISSING=false

And at Line 294 to 302 you set this to "true" if you find a php.ini without a correct date.timezone. Right?

So, then in my opinion Line 588 should be:

if [ ${PHPINITIMEZONEMISSING} = true ]

Otherwise the If Statement is everytime true.

Greetings

My System: Ubuntu Server 16.04.6 x64

wolf-in-sheeps-clothing avatar Aug 20 '19 19:08 wolf-in-sheeps-clothing

Good catch! Thank you.

widhalmt avatar Aug 22 '19 10:08 widhalmt