Use detail property rather than PerformanceEntry

[dennypradipta]

In the newest build of Monika, there are deprecation warnings:

2024-07-22 14:02:39.028 | (node:1) [DEP0152] DeprecationWarning: Custom PerformanceEntry accessors are deprecated. Please use the detail property. 
2024-07-22 14:02:39.029 | (Use `node --trace-deprecation ...` to show where the warning was created)

This should be replaced with detail property rather than use PerformanceEntry

[suyono3484]

I did a quick check. It looks like the PerformanceEntry comes from one of the dependencies (or indirect dependencies).

How did I check? I ran this command in the root of the source directory: grep -inr "performanceentry" * | grep -v "js.map". The command only yields results after I run npm ci and npm run build --workspaces --if-present. Using a clean source tree yielded no results.

[dennypradipta]

The error originally was found on the Grafana for Symon:

I'm guessing that this happened only in Monika Docker images, as Symon uses dockerized version. I could be wrong though, but the logs point out that there is a command that you can run to detect it:

node --trace-deprecation ...

[suyono3484]

I pulled the latest docker image of monika from the docker hub. I couldn't reproduce the deprecation message. @dennypradipta, could you share the config where monika produces the deprecation message?

[suyono3484]

@dennypradipta shared the configuration where Monika produced the deprecation message with me, but I still couldn't reproduce the message using docker locally. After some discussion, the message might be logged when Monika runs in Symon mode. I am reassigning this issue to @dennypradipta since he has more knowledge related to Symon.