-
Notifications
You must be signed in to change notification settings - Fork 55
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
Add InterceptResponse to the Plugin interface #205
Conversation
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## main #205 +/- ##
==========================================
- Coverage 71.10% 70.99% -0.11%
==========================================
Files 26 26
Lines 2710 2717 +7
==========================================
+ Hits 1927 1929 +2
- Misses 660 665 +5
Partials 123 123
☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks fine to me, maybe want to add an example of some ways you can use this (maybe doing what that other person was asking about)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Agree with @LeonHuston comment about examples, we should probably have one for each plugin method.
Changes look good.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you squash the typo commit though @nmaquet
InterceptResponse is called after bramble has finished executing a request. It can be used to inspect and/or modify the response bramble will return.
21d7cd9
to
8bba076
Compare
Done! |
InterceptResponse is called after bramble has finished executing a request. It can be used to inspect and/or modify the response bramble will return.
Added to help with #204