search

vercel / speed-insights

Vercel Speed Insights package
https://vercel.com/docs/speed-insights
Apache License 2.0
57 stars 9 forks source link

Fix speed insights sdk edge cases for dynamic route generation #15

Closed tobiaslins closed 10 months ago

tobiaslins commented 10 months ago

📓 What's in there?

Our current generation breaks when ( and similar are in the URL. Fix those cases and make sure it can never break

Customer issue: https://vercel.slack.com/archives/C02UZSM1CP6/p1700495470466899

🧪 How to test?

Run the tests

linear[bot] commented 10 months ago
ALY-1068 Fix speed-insights sdk edge cases

Currently speed insights crashes when the regex is invalid [https://vercel.slack.com/archives/C02UZSM1CP6/p1700495470466899](https://vercel.slack.com/archives/C02UZSM1CP6/p1700495470466899)

vercel[bot] commented 10 months ago

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

Name Status Preview Comments Updated (UTC)
speed-insights-nextjs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 20, 2023 4:39pm
speed-insights-remix ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 20, 2023 4:39pm
speed-insights-sveltekit ✅ Ready (Inspect) Visit Preview Nov 20, 2023 4:39pm