Fix variable value in Response.isOnline (ApiMonitor and WebsiteMonitor) #457

Closed
opened 2026-04-05 16:19:53 +02:00 by MrUnknownDE · 0 comments
Owner

Originally created by @Vincent-Bouffard-Ouidou on 6/2/2025

Today one of my website was down and the response was a timeout.

But no incident created because my criteria are :

  • Check if "Is Online" is false
    or
  • Check if "Response Status Code" is not equal to 200

But the website was not considered as "Offline" and i think it's because the bad value is present in code :

https://github.com/OneUptime/oneuptime/blob/master/Probe/Utils/Monitors/MonitorTypes/ApiMonitor.ts#L200
https://github.com/OneUptime/oneuptime/blob/master/Probe/Utils/Monitors/MonitorTypes/WebsiteMonitor.ts#L195

For me, for this line :

  • apiResponse.isOnline = true; => apiResponse.isOnline = false;
  • probeWebsiteResponse.isOnline = true; => probeWebsiteResponse.isOnline = false;

Thanks.

*Originally created by @Vincent-Bouffard-Ouidou on 6/2/2025* Today one of my website was down and the response was a timeout. But no incident created because my criteria are : - Check if "Is Online" is false or - Check if "Response Status Code" is not equal to 200 But the website was not considered as "Offline" and i think it's because the bad value is present in code : https://github.com/OneUptime/oneuptime/blob/master/Probe/Utils/Monitors/MonitorTypes/ApiMonitor.ts#L200 https://github.com/OneUptime/oneuptime/blob/master/Probe/Utils/Monitors/MonitorTypes/WebsiteMonitor.ts#L195 For me, for this line : - `apiResponse.isOnline = true;` => `apiResponse.isOnline = false;` - `probeWebsiteResponse.isOnline = true;` => `probeWebsiteResponse.isOnline = false;` Thanks.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github/oneuptime#457