-
Notifications
You must be signed in to change notification settings - Fork 306
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
Automated user events #3205
Automated user events #3205
Conversation
* Config variables. * Track event modifications. * Passport verify channel handler.
Overall package sizeSelf size: 4.72 MB Dependency sizes
🤖 This report was automatically generated by heaviest-objects-in-the-universe |
Codecov Report
@@ Coverage Diff @@
## master #3205 +/- ##
==========================================
+ Coverage 85.88% 86.00% +0.12%
==========================================
Files 199 200 +1
Lines 7692 7760 +68
Branches 33 33
==========================================
+ Hits 6606 6674 +68
Misses 1086 1086
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
…g/dd-trace-js into julio/automated-user-events-local
…tomated-user-events-local
f3676a3
to
1595f8b
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.
You should update test.ts
and index.d.ts
files to add configuration options.
ea3ddd2
to
f7338c8
Compare
79044b3
to
4eed545
Compare
* Add passport local and http instrumentations. * Add automated event tracking in appsec.. --------- Co-authored-by: Carles Capell <107924659+CarlesDD@users.noreply.github.com> Co-authored-by: simon-id <simon.id@datadoghq.com>
* Add passport local and http instrumentations. * Add automated event tracking in appsec.. --------- Co-authored-by: Carles Capell <107924659+CarlesDD@users.noreply.github.com> Co-authored-by: simon-id <simon.id@datadoghq.com>
* Add passport local and http instrumentations. * Add automated event tracking in appsec.. --------- Co-authored-by: Carles Capell <107924659+CarlesDD@users.noreply.github.com> Co-authored-by: simon-id <simon.id@datadoghq.com>
* Add passport local and http instrumentations. * Add automated event tracking in appsec.. --------- Co-authored-by: Carles Capell <107924659+CarlesDD@users.noreply.github.com> Co-authored-by: simon-id <simon.id@datadoghq.com>
* Add passport local and http instrumentations. * Add automated event tracking in appsec.. --------- Co-authored-by: Carles Capell <107924659+CarlesDD@users.noreply.github.com> Co-authored-by: simon-id <simon.id@datadoghq.com>
* Add passport local and http instrumentations. * Add automated event tracking in appsec.. --------- Co-authored-by: Carles Capell <107924659+CarlesDD@users.noreply.github.com> Co-authored-by: simon-id <simon.id@datadoghq.com>
* Add passport local and http instrumentations. * Add automated event tracking in appsec.. --------- Co-authored-by: Carles Capell <107924659+CarlesDD@users.noreply.github.com> Co-authored-by: simon-id <simon.id@datadoghq.com>
* Add passport local and http instrumentations. * Add automated event tracking in appsec.. --------- Co-authored-by: Carles Capell <107924659+CarlesDD@users.noreply.github.com> Co-authored-by: simon-id <simon.id@datadoghq.com>
* Add passport local and http instrumentations. * Add automated event tracking in appsec.. --------- Co-authored-by: Carles Capell <107924659+CarlesDD@users.noreply.github.com> Co-authored-by: simon-id <simon.id@datadoghq.com>
* Add passport local and http instrumentations. * Add automated event tracking in appsec.. --------- Co-authored-by: Carles Capell <107924659+CarlesDD@users.noreply.github.com> Co-authored-by: simon-id <simon.id@datadoghq.com>
* Add passport local and http instrumentations. * Add automated event tracking in appsec.. --------- Co-authored-by: Carles Capell <107924659+CarlesDD@users.noreply.github.com> Co-authored-by: simon-id <simon.id@datadoghq.com>
What does this PR do?
This PR add passport-local and passport-http instrumentation in order to set the appropriate events tags in the request. This is achieved by creating a new channel and publishing the information obtained during the authentication process.
Additional notes
This PR add hooks for passport-http and passport-local libraries. Specifically we are instrumenting the constructors in order to get the results from the verified callback which is called by the authenticate method in Passport. Files involved: