YOURLS does wrong comparisons for the configuration about images

[LeoColomb]

Thanks for this report, @nymous!
What is your config.php?
Instead of editing core file, you should prefer using the config file.
The one distributed with the Docker image is written to respect environnement variables.

https://github.com/YOURLS/docker-yourls/blob/d076c5dbc3d1641e855765c3410490e710b19b6f/config-docker.php#L43-L51

[nymous]

I didn't change the config.php from the Docker image, that isn't the issue here.

Environment variables are respected, but they are retrieved as strings, which in YOURLS code are compared to a boolean. In PHP, the string "false" is truthy.

My patch forces YOURLS to convert the constant value to the corresponding boolean, whether this value is already a boolean or a string.

That being said, maybe this isn't YOURLS responsibility to handle this better, maybe the maintainers from the Docker image should patch it, as it works if the config file is edited manually to enter a boolean value...

[nymous]

Wait, did I really create this issue in the Docker repository? Woops, it was supposed to go to the YOURLS source repository, sorry! 😅

[LeoColomb]

Wait, did I really create this issue in the Docker repository?

No, I moved it, the issue does not target YOURLS itself but its usage inside the docker image.

Environment variables are respected, but they are retrieved as strings, which in YOURLS code are compared to a boolean. In PHP, the string "false" is truthy

OK, got it now.
What about fixing this on the PHP constants in the config.php directly?

[nymous]

Oh, right. Thought I lost my mind for a second ^^

Well, adding this filter in the config file seems like a better idea. Not sure if there is any other variable that would be affected by the same issue.

[LeoColomb]

Agreed. I think this is the case for all boolean variables.
Would you aim opening a pull request with the fix? 👍