Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Cert Exp not shown in overview, but service marked as down when expired #3551

Closed
2 tasks done
liquidat opened this issue Aug 9, 2023 · 11 comments · Fixed by #4700 · May be fixed by #4281
Closed
2 tasks done

Cert Exp not shown in overview, but service marked as down when expired #3551

liquidat opened this issue Aug 9, 2023 · 11 comments · Fixed by #4700 · May be fixed by #4281
Labels
area:cert-expiry related to certificate expiry notifications area:monitor Everything related to monitors bug Something isn't working

Comments

@liquidat
Copy link

liquidat commented Aug 9, 2023

⚠️ Please verify that this bug has NOT been raised before.

  • I checked and didn't find similar issue

🛡️ Security Policy

Description

One out of my many tests does show the "Cert Exp." in the monitor overview - although it showed the service as DOWN when the certificate expired!

Screenshot of the service (my local FritzBox):
image
Screenshot of the same configuration, for my local Grafana, as comparison:
image

Monitor configuration:
image

Logs in UI, showing that the monitor was down:
image

Logs indicating that the expiry date was correctly identified, see below.

👟 Reproduction steps

  1. Set up Kuma
  2. Set up FritzBox with certificate (let's encrypt, domain based)
  3. Set up Kuma monitor on FritzBox, activate "Certificate Expiry Notification"

👀 Expected behavior

The cert expiry should be shown in the monitor overview.

😓 Actual Behavior

The cert expiry is not shown.

Note that in the backed everything is working fine: when we reached the expiry date, the service was shown as down in Kuma!

🐻 Uptime-Kuma Version

1.22.1

💻 Operating System and Arch

Raspbian GNU/Linux 11 (bullseye), ARM

🌐 Browser

Chrome, Firefox

🐋 Docker Version

Docker version 20.10.5+dfsg1, build 55c4c88

🟩 NodeJS Version

16

📝 Relevant log output

uptime-kuma     | 2023-08-09T13:09:12+02:00 [MONITOR] WARN: Monitor #14 'FritzBox': Failing: certificate has expired | Interval: 60 seconds | Type: http | Down Count: 0 | Resend Interval: 0
uptime-kuma     | 2023-08-09T13:10:12+02:00 [MONITOR] WARN: Monitor #14 'FritzBox': Failing: certificate has expired | Interval: 60 seconds | Type: http | Down Count: 0 | Resend Interval: 0
uptime-kuma     | 2023-08-09T13:11:12+02:00 [MONITOR] WARN: Monitor #14 'FritzBox': Failing: certificate has expired | Interval: 60 seconds | Type: http | Down Count: 0 | Resend Interval: 0
@liquidat liquidat added the bug Something isn't working label Aug 9, 2023
@zappityzap
Copy link
Contributor

I've seen this with one of my monitors too, but haven't been able to figure out why.

@CommanderStorm CommanderStorm added area:monitor Everything related to monitors area:cert-expiry related to certificate expiry notifications labels Dec 5, 2023
@xifengzhu
Copy link

I've seen this with one of my monitors too.

@chakflying
Copy link
Collaborator

Would be great if someone can share a url where this is reproducible so we can test and debug this.

@xifengzhu
Copy link

Would be great if someone can share a url where this is reproducible so we can test and debug this.

https://wordmind.ai

@CommanderStorm
Copy link
Collaborator

CommanderStorm commented Dec 26, 2023

@xifengzhu
Cannot reproduce; The cert is valid and is displayed as such.
The cert expiry @liquidat reported is also displayed.
Am I misssing something?

image

@xifengzhu
Copy link

xifengzhu commented Dec 26, 2023

@xifengzhu Cannot reproduce; The cert is valid and is displayed as such. The cert expiry @liquidat reported is also displayed. Am I misssing something?

image

Cursor
Cursor

I found that https://wordmind.ai can recognize the certificate, but if it is https://wordmind.ai/health_check, it cannot recognize the certificate.

@Niemand2006
Copy link

This error is when the Port from https is not 443.

@ihcops
Copy link

ihcops commented Mar 9, 2024

Would be great if someone can share a url where this is reproducible so we can test and debug this.

https://bill.vps1.net/

20240309174i308

@ihcops
Copy link

ihcops commented Mar 9, 2024

Would be great if someone can share a url where this is reproducible so we can test and debug this.

https://bill.vps1.net/

sqlite> select * from monitor_tls_info where monitor_id=45;
17|45|{"valid":true,"certInfo":null}

@CommanderStorm
Copy link
Collaborator

Tested #4281 as working and fixing this example too

Details

image

@CommanderStorm
Copy link
Collaborator

Resolved in next patch release

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:cert-expiry related to certificate expiry notifications area:monitor Everything related to monitors bug Something isn't working
Projects
None yet
7 participants