Skip to content
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

Added record functionality #38

Closed
wants to merge 4 commits into from
Closed

Added record functionality #38

wants to merge 4 commits into from

Conversation

framp
Copy link

@framp framp commented Oct 22, 2016

This pull request adds support for recording requests, piggybacking on mitm.js support for intercepting net calls.

The data is going to be emitted via a record event.

@framp
Copy link
Author

framp commented Oct 23, 2016

@moll I'm thinking of using this feature in another library, so I'd love any inputs on the exposed API

@framp framp mentioned this pull request Nov 25, 2016
@framp
Copy link
Author

framp commented Nov 25, 2016

No answer, so I will publish the project under a different name.

Happy to have a discussion about this - feel free to merge #40

@framp framp closed this Nov 25, 2016
@moll
Copy link
Owner

moll commented Nov 25, 2016

Hey! Thanks and absolutely, get it going under your own name and then once I'm back on a PR review roll, we'll see if something can be united. :)

@gr2m gr2m mentioned this pull request Dec 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants