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.

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

backports/v1.0: redaction: fix crash related to redaction filters + improve performance #2324

Merged
merged 1 commit into from
Apr 11, 2024

Commits on Apr 11, 2024

  1. redaction: fix crash related to redaction filters + improve performance

    [upstream commit: 91012a8]
    
    The previous redaction filters implementation was unsound, as it made modifications to the
    process object within an event message, but this object was already being shared in the
    event cache. This caused a data race that in turn resulted in a frequent crash when
    applying redaction filters on a modest workload. Moreover, the prior implementation was
    not very performant, since it relied on iterating over all fields in an event message to
    redact strings. In practice, we only really care about arguments in the majority of use
    cases.
    
    To address the above issues, we make a small breaking change in how the redaction filters
    work to focus ONLY on arguments and apply them much earlier in the pipeline. This both
    fixes the crash and significantly reduces performance impact, while solving the primary
    use case.
    
    Signed-off-by: William Findlay <[email protected]>
    willfindlay committed Apr 11, 2024
    Configuration menu
    Copy the full SHA
    692d42a View commit details
    Browse the repository at this point in the history