-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
Finish implementing trace anonymizer #2556
Comments
Hi @yurishkuro @jpkrohling may I pick and try solving this issue as two PRs for the two parts. |
It's yours! |
Hi @yurishkuro This PR implements only the first part of the issue, second part (main program) is still remaining, which I am planning to pick next. Sorry for creating the confusion. |
Yes, I know - github auto-closed the issue because the PR had a magic clause "fixes ####". |
Sorry my bad, I shouldn't have added fixes ## thing in the PR. |
Requirement - what kind of business use case are you trying to solve?
Allow people to safely export production traces into GitHub tickets (as a way of reproducing problems, especially with the UI) without exposing sensitive information.
Problem - what in Jaeger blocks you from solving the requirement?
The anonymizer is partially implemented.
Proposal - what do you suggest to solve the problem or improve the existing situation?
Finish the implementation as follows
allowsTags
are also hashed, when false they are preserved as iscc @jpkrohling
The text was updated successfully, but these errors were encountered: