-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Expose Binlog reader API needed for sensitive data redacting #9132
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.
Am I correct in saying this PR doesn't actually do anything, just sets us up to be able to redact logs more easily in the future as they're produced?
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.
Looks good to me! I'm excited to see this getting used!
133322d
to
36add10
Compare
rekicking CLA bot |
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
Contributes to #9089
Context
The simpliest way of redacting specific strings from binlogs is by replaying it via reader, while altering strings and immediately writing to new binlog via writer. To allow this scenario - the string manipulation during reading is exposed.
Based on team decision - the actual leveraging of the exposed functionality will currently live outside of this repo
Changes Made
Span<char>
as an event arg - allowing in-place changes - but that would mean we'd need to create the span [which involves copying of the internal string buffer], for each read string, even those we wouldn't need to alter)