bug: ping shows green color on error status codes

[MartinLoeper]

Describe the bug

I configured ping for my applications and they display the status code as expected when hovering. However, the color does not change to red once a non-200 status code is received.

Steps to reproduce

• configure a ping for an app
• make the ping fail by e.g. changing the ping url to a 404 page
• check the color of the ping indicator on your board

Impact

The ping indicator on each app tile has limited usefulness because I have to hover it to see the status code to determine whether the app is alive.

Additional information

Version

1.0.0-beta

Installation method

Docker Run

Browser

Firefox

[manuel-rw]

This is expected behavior. The "ping" functionality was never intended to check for status codes. Back in 2022, this feature was introduced without much discussion around it. Homarr's purpose is not to monitor the uptime of your apps or whether they are healthy. The name "ping" also implies that only the connection is tested - which was untrue in <1.0 but now true in 1.0.
I understand your desire though and we are planning to add proper monitoring after the public release of 1.0. We will add support for Uptime Kuma (and more similar tools in iterative steps, see ajnart/homarr#2241), which will enable you to monitor much more than just HTTP(-S) links. Uptime Kuma can also monitor databases, TCP connections, popular game servers, docker containers and more! Most of our users already use it anyway, so it's a good idea to re-use and integrate it into Homarr - like we do with many other integrations.

As a summary; all status codes will be displayed as "green" in 1.0 any only connections that could not be established will be shown as "red". Support for better monitoring integration(-s) will follow.

[MartinLoeper]

Understandable. Thanks for the clarification @manuel-rw. It is not a crucial feature for me anyway. I was just surprised as the color and the status code tooltip suggested to me that there might be a status code check intended. which is not the case as you explained