Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[Core] Sanitize query params in url tracing attribute #35546

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

[Core] Sanitize query params in url tracing attribute #35546

wants to merge 4 commits into from

Commits on May 22, 2024

  1. [Core] Sanitize query params in url tracing attribute 

    Query params in URLs printed in HTTP span attributes are now sanitized
by default.

A new keyword argument is also added to the constructor of
DistributedTracingPolicy to enable users to customize what query
parameters are allowed.

Signed-off-by: Paul Van Eck <paulvaneck@microsoft.com>
    @pvaneck
    pvaneck committed May 22, 2024
    Configuration menu
    Copy the full SHA
    b89259a View commit details
    Browse the repository at this point in the history

Commits on May 23, 2024

  1. Update util function 

    Signed-off-by: Paul Van Eck <paulvaneck@microsoft.com>
    @pvaneck
    pvaneck committed May 23, 2024
    Configuration menu
    Copy the full SHA
    a77ffbd View commit details
    Browse the repository at this point in the history

  2. update docstring 

    Signed-off-by: Paul Van Eck <paulvaneck@microsoft.com>
    @pvaneck
    pvaneck committed May 23, 2024
    Configuration menu
    Copy the full SHA
    8021049 View commit details
    Browse the repository at this point in the history

  3. Minor adjustments based on feedback 

    Signed-off-by: Paul Van Eck <paulvaneck@microsoft.com>
    @pvaneck
    pvaneck committed May 23, 2024
    Configuration menu
    Copy the full SHA
    57a749e View commit details
    Browse the repository at this point in the history