Domain name xxxxx will expire in NaN days

[ausebiblibre]

📑 I have found these related issues/pull requests

Didn't any related issue

🛡️ Security Policy

• I have read and agree to Uptime Kuma's Security Policy.

📝 Description

Since update to 2.1.0-beta1 I receive alerts about domains expiration.
I checked the database and the columns in the domain_expiry table are already filled, but in fact it looks like I received about ALL monitors I configured.

👟 Reproduction steps

...

👀 Expected behavior

Notifications are set to 7, 14 and 21 days.

😓 Actual Behavior

I receive alerts about domains that won't expire before several months.

🐻 Uptime-Kuma Version

2.1.0-beta1

💻 Operating System and Arch

Ubuntu 24.04 LTS
MariaDB 11

🌐 Browser

any

🖥️ Deployment Environment

deployed in LXC container under Ubuntu 24.04 LTS with mariadb. Upgraded from 2.0.0-beta.3

📝 Relevant log output

[CommanderStorm]

Could you give me one of these domains?
My email is in my profile.

I don't know why you are getting NaN for domain expiry.
Maybe the RDAP endpoint is for that one is wrong, would be good to debug before we fix the wrong thing

[CommanderStorm]

CC @shanto

[ausebiblibre]

Just sent you an email with a list ;)

[CommanderStorm]

I don't see any, you sure?

[ausebiblibre]

I don't see any, you sure?

sure, sent 2 ;) Will send an other one with my private email

[ausebiblibre]

I don't know why you are getting NaN for domain expiry. Maybe the RDAP endpoint is for that one is wrong, would be good to debug before we fix the wrong thing

Please let me know if I can add some trace in any file.

[CommanderStorm]

hard to debug for me. The domains you send don't behave like you reported.

Not sure if I just applied a bandaid fix to prevent those symptoms, but now we have a log which gives more information and don't send out notifications.
Could you try to look into the log once we release our next beta?

[ausebiblibre]

Sure I'll do that ! I forgot a detail I just see now : the domain link (and title) both are written like ////domain.com

[CommanderStorm]

that is also weird.
So might be that the expiry notifications don't entirely work reliably out of the box and need another round.
We will see

[ausebiblibre]

As said in my last email, the issue is that there's no check on date format in server/model/util-server.js in the getDaysBetween() function, so it happens to try to calculate days between carrots and potatoes.