-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Labels
Bug (unconfirmed)issues that could not be reproduced yetissues that could not be reproduced yetDocker 🐋Everything related to DockerEverything related to DockerGood first issue 1️⃣Beginner friendly - New contributors are welcomeBeginner friendly - New contributors are welcomeSecurity 🛡️help wanted
Milestone
Description
Describe the bug
Current healthcheck logic doesn't work with OIDC.
Here is the original checks:
if ($httpCode !== 200 || !is_string($content) || !str_contains($content, 'jsonVars') || !str_contains($content, '</html>')) {For OIDC:
httpCodewill be 302 (moved)contentdoesn't contain "json"
This line will work:
if (($httpCode !== 302 && $httpCode !==200) || !is_string($content) || !str_contains($content, '</html>')) {To Reproduce
- Enable OIDC
- Run
cli/health.php - Get exit code (
echo $?)
Expected behavior
Script exit code should be 0
FreshRSS version
1.27.1
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
Bug (unconfirmed)issues that could not be reproduced yetissues that could not be reproduced yetDocker 🐋Everything related to DockerEverything related to DockerGood first issue 1️⃣Beginner friendly - New contributors are welcomeBeginner friendly - New contributors are welcomeSecurity 🛡️help wanted