-
Notifications
You must be signed in to change notification settings - Fork 142
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
✨ [RUMF-772] add beforeSend API #644
Conversation
723d60a
to
5604edd
Compare
Codecov Report
@@ Coverage Diff @@
## master #644 +/- ##
==========================================
+ Coverage 87.50% 87.60% +0.10%
==========================================
Files 54 55 +1
Lines 2464 2510 +46
Branches 509 517 +8
==========================================
+ Hits 2156 2199 +43
- Misses 308 311 +3
Continue to review full report at Codecov.
|
f5171f3
to
1aa110a
Compare
f42a5d8
to
578f648
Compare
e01cc6d
to
a4f0e8b
Compare
dbf61de
to
848e0b4
Compare
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 great!
0118968
to
848e0b4
Compare
53c445a
to
73c7402
Compare
Motivation
Provide an API to modified fields that can contain sensitive data before sending them to our server:
ex:
Only the following fields are updatable:
view.url
view.referrer
action.target.name
error.message
error.stack
error.resource.url
resource.url
view.url
view.referrer
message
error.stack
http.url
Changes
Testing
automated tests + sandbox tests
I have gone over the contributing documentation.