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

Serilog enrichments. #2597

Merged
merged 7 commits into from
Jan 13, 2020
Merged

Serilog enrichments. #2597

merged 7 commits into from
Jan 13, 2020

Conversation

mehmetuken
Copy link
Contributor

Resolve #831

  • Adding Serilog enrichments middleware. Adding this properties to LogContext TenantId, UserId, ClientId, CorrelationId.

Using WriteTo.File output template
{Timestamp:yyyy-MM-dd HH:mm:ss.fff zzz} [{Level:u3}] {Message:lj} {Properties}{NewLine}{Exception}
or yours using Sinks.MSSqlServer, Sinks.Elasticsearch auto save properties.

Change property names set AbpAspNetCoreSerilogEnrichersOptions.

  • Change Serilog.Extensions.Hosting packages to Serilog.AspNetCore
    Serilog.Extensions.Hosting says:

ASP.NET Core applications should consider using Serilog.AspNetCore instead, which bundles this package and includes other ASP.NET Core-specific features.

@maliming maliming added this to the 2.1 milestone Jan 9, 2020
@maliming maliming requested review from hikalkan and maliming January 9, 2020 08:35
@maliming
Copy link
Member

maliming commented Jan 9, 2020

Thanks @mehmetuken awesome! 💯

Resolved #2583 issue at the same time.

@hikalkan hikalkan modified the milestones: 2.1, 2.0 Jan 9, 2020
@hikalkan
Copy link
Member

hikalkan commented Jan 9, 2020

Great @mehmetuken I will review this.

@hikalkan hikalkan merged commit f31c335 into abpframework:dev Jan 13, 2020
hikalkan added a commit that referenced this pull request Jan 13, 2020
…n align the module name and the package name.
hikalkan added a commit that referenced this pull request Jan 13, 2020
@mehmetuken mehmetuken deleted the serilog-enrichment branch January 13, 2020 13:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Volo.Abp.AspNetCore.Serilog Package
3 participants