Skip to content

Conversation

dnoakes
Copy link
Contributor

@dnoakes dnoakes commented Jul 23, 2024

Example usage,

Serilog: Log.ForContext("Tags", new List() { "Tag1", "Tag2"}).Information("Seri info");
ILogger:
using (var scope = _logger.BeginScope(new Dictionary<string, object> { ["Tags"] = new string[] { "Test1", "Test2" }}))
{
_logger.Log(logLevel, eventId, state, exception, formatter);
}

image

@CLAassistant
Copy link

CLAassistant commented Jul 23, 2024

CLA assistant check
All committers have signed the CLA.

@niemyjski
Copy link
Member

niemyjski commented Jul 24, 2024

Thanks for the PR! I've left a comment on the tags implementation. I wonder if it's worth supporting other first-class properties like value as well.

@niemyjski niemyjski merged commit d71d3f8 into exceptionless:master Jul 24, 2024
@niemyjski
Copy link
Member

Thanks again for the PR. I just published https://github.com/exceptionless/serilog-sinks-exceptionless/releases/tag/v5.0.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

3 participants