-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
[network-plugin] Feat: Capture network events (2) #1689
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
🦋 Changeset detectedLatest commit: 55bc747 The changes in this PR will be included in the next version bump. This PR includes changesets to release 0 packagesWhen changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
3b65956
to
1fdcbdc
Compare
5223ce4
to
9c75fef
Compare
9c75fef
to
d3b4c9f
Compare
hey @crutch12 answering from here #1105 (comment) i'm not a core contributor to rrweb, so i can't merge it and release it iirc there were some things we had to change at PostHog while using this. you can see our current implementation here https://github.com/PostHog/posthog-js/blob/main/src/extensions/replay/external/network-plugin.ts you're welcome to copy some, all, or none of that - it's MIT licensed and based on the original PR here originally anyway the next step for us is to make this data queryable and I'll probably move it out of the replay ingestion flow for us anyway... i did intend to contribute this back but just never made time with the other plates i'm spinning :) |
I've adapted old pull request for new plugins folder structure. I hope @jlalmes doesn't mind 🙏
See the description here: #1105
Closes #552