-
Notifications
You must be signed in to change notification settings - Fork 617
Update active-tracing.md #8793
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: main
Are you sure you want to change the base?
Update active-tracing.md #8793
Conversation
Add logs
✅ Deploy Preview for kongdocs ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
||
When initiating a tracing session, administrators can choose to enable log capture, which collects detailed Kong Gateway logs for the duration of the session. These logs are then correalted with traces using *trace_id* and *span_id* providing a comprehensive and drill-down view of logs generated during specific trace or span. | ||
|
||
## Payload Capture |
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.
are we going to hold off on documenting this feature until we are sure it can make it to GA? (Payload capture)
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.
Yeah, my plan is to keep it ready in case it makes it
|
||
## Key highlights | ||
Log sanitizer uses Luhn algorithm, a well-known algorithm to validate credit card numbers, International Mobile Equipment Identity (IMEI) numbers, and other numerical data. The redaction is done by replacing the matched characters with * |
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.
not sure if we should mention the IMEI since we are not doing that redaction (yet)
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.
ok, will remove
typo fix accepted Co-authored-by: Samuele <[email protected]>
Updated
Updated/removed extra line
Minor updates to wordings
Minor updates
Add logs
Description
Adding docs for Active Tracing logs
https://konghq.atlassian.net/browse/KAG-7049
Testing instructions
Preview link:
Checklist