-
-
Notifications
You must be signed in to change notification settings - Fork 308
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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: sanitize span data #1963
fix: sanitize span data #1963
Conversation
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.
Please remove the sanitation for the tags. Then LGTM 馃殌 and we can merge.
`Instructions and example for changelog`$
To the changelog entry, please add a link to this PR (consider a more descriptive message):` - sanitize span data(#1963)
|
Uh, it has a changelog entry. Weird. |
* master: fix: sanitize span data (#1963)
馃摐 Description
Sanitize span data when serializing.
馃挕 Motivation and Context
Closes #1307
馃挌 How did you test it?
Unit tests
馃摑 Checklist
馃敭 Next steps