PostHog / posthog-js

Send usage data from your web app or site to PostHog, with autocapture.
https://posthog.com/docs/libraries/js
Other
278 stars 114 forks source link

fix: captured network requests must always have name property #1398

Closed pauldambra closed 3 weeks ago

pauldambra commented 3 weeks ago

v1.160.2 introduced much more robust network capture... too robust because now we can capture requests without their URL. no bueno

let's always capture the URL so that if there is no performance entry we at least have that url

(for the benefit of the future traveller... the requests captured without a URL are requests that we previously couldn't capture at all)

vercel[bot] commented 3 weeks ago

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

Name Status Preview Updated (UTC)
posthog-js ✅ Ready (Inspect) Visit Preview Sep 3, 2024 2:57pm
github-actions[bot] commented 3 weeks ago

Size Change: +123 B (+0.01%)

Total Size: 1.18 MB

Filename Size Change
dist/array.full.js 336 kB +41 B (+0.01%)
dist/recorder-v2.js 110 kB +41 B (+0.04%)
dist/recorder.js 111 kB +41 B (+0.04%)
ℹ️ View Unchanged | Filename | Size | | :--- | :---: | | `dist/array.js` | 156 kB | | `dist/exception-autocapture.js` | 10.4 kB | | `dist/main.js` | 157 kB | | `dist/module.js` | 156 kB | | `dist/surveys-preview.js` | 59.8 kB | | `dist/surveys.js` | 66 kB | | `dist/tracing-headers.js` | 8.26 kB | | `dist/web-vitals.js` | 5.79 kB |

compressed-size-action

marandaneto commented 3 weeks ago

we can capture requests without their URL. no bueno

😅