getsentry / getsentry/sentry-javascript

Always attach some URL/target data point to fetch/xhr/request errors

未关闭
#13,168 0 条评论 1 个 reaction 已指派 0 人 在 GitHub 查看
Browser Errors Improvement
主要语言
TypeScript
星标
8.7k
派生
1.8k
平均合并
1 天 17 小时
30 天内合并 PR
515

描述

### Description

The Sentry Chief Product Officer has brought to our attention that Fetch errors sometimes suck. ([example that will expire in 90 days](https://peated.sentry.io/issues/5656229682/?referrer=regression_activity-email¬ification_uuid=add413c0-019a-4b87-ae37-608eb905a605))

![Image](https://github.com/user-attachments/assets/a8106189-e78e-486f-b5c8-5373af5cc61d)

We technically have a stack trace here but sourcemaps are broken so the user has basically no idea what fetch request failed. It's very hard to gauge from breadcrumbs what exact request failed.

Side note, the fetch call that failed looked like `fetch("")`. We render this as in the product, you guessed it `""`. Which is crap because it actually means the request went to the current route. We should at least coerce that string to `URL(fetchTarget, window.location.href).toString()` in the SDK.

![Image](https://github.com/user-attachments/assets/2deb0b10-8afd-4efb-8171-592eb98e4859)

贡献指南

打开贡献指南

评估

这个 Issue 还没有评估数据。

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。