What/Why:
I just noticed that the startTime to record the duration of the execute() function has an error in that this is an async function.
I also think this metric function_duration_milliseconds can have a better name, as in more explicit about it being about a single function in one service.
Also, I need to reinstall my MW local env bc it's due for an update but I'm running into weird behavior with bypass-cache? Not the new toggle itself per se (?) but I usually set it to true (disabling it) in my local env for testing purposes. However, it seems to cache results regardless so I can't see fresh data from new triggers from the UI, even if I restart my docker...
How:
- use try/finally block inside async function execute()