catch and swallow exceptions on fetch

[schehata]

while sending logs exceptions could happend, swallowing the exception ensures we don't interrupts customer apps or break api functions responses. We do the same in web-vitals. I am not sure though about removing await, need to think about it and do some testing.

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated
next-axiom-example	✅ Ready (Inspect)	Visit Preview	💬 Add your feedback	Jan 27, 2023 at 10:08AM (UTC)

[oranoran]

LGTM.

Regarding the idea of removing await - from previous testing I've done on Vercel, it looks like it's ok to keep running after sending a response. The only limit to this is the function timeout.
I didn't test this on platforms other than Vercel.

[bahlo]

i'm a bit concerned about deliverability if we don't await, otherwise nice work ✌️

[schehata]

LGTM.

Regarding the idea of removing await - from previous testing I've done on Vercel, it looks like it's ok to keep running after sending a response. The only limit to this is the function timeout. I didn't test this on platforms other than Vercel.

would the function wait for the fetch to finish if we don't await ? that's basically our concern.

[schehata]

removing await seems not to change anything, the logs still arrives safely, but I am not sure about the execution time yet, but it seems to be better than sending the logs with await. will merge this for now, and continue testing other scenarios.