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
Adding Microsoft.Extensions.Telemetry README #4671
Conversation
src/Libraries/Microsoft.Extensions.Compliance.Redaction/README.md
Outdated
Show resolved
Hide resolved
Co-authored-by: Chris Ross <Tratcher@Outlook.com> Co-authored-by: Sébastien Ros <sebastienros@gmail.com>
options.ApplyDiscriminator = true; | ||
}); | ||
|
||
builder.Services.AddRedaction(); // <- This call is required in order for the redactor provider to be added into the service collection. |
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.
IIRC this call requires Microsoft.Extensions.Compliance.Redaction
to be referenced
|
||
```csharp | ||
// Enable log enrichment. | ||
builder.EnableEnrichment(options => |
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.
The extension method should be applied on an ILoggingBuilder
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.
Same applies to EnableRedaction()
call
Adding README for Microsoft.Extensions.Telemetry and making one small clarification on one of the Compliance READMEs to explain how RedactorProviders should be used.
Microsoft Reviewers: Open in CodeFlow