Skip to content

Commit 8d4a722

Browse files
committed
Follow-up from #265 - don't include query in logged request path by default; bump major version
1 parent 83043f2 commit 8d4a722

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

src/Serilog.AspNetCore/AspNetCore/RequestLoggingOptions.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -70,9 +70,9 @@ static LogEventLevel DefaultGetLevel(HttpContext ctx, double _, Exception ex) =>
7070

7171
/// <summary>
7272
/// Include the full URL query string in the <c>RequestPath</c> property
73-
/// that is attached to request log events. The default is <c>true</c>.
73+
/// that is attached to request log events. The default is <c>false</c>.
7474
/// </summary>
75-
public bool IncludeQueryInRequestPath { get; set; } = true;
75+
public bool IncludeQueryInRequestPath { get; set; }
7676

7777
/// <summary>
7878
/// Constructor

src/Serilog.AspNetCore/Serilog.AspNetCore.csproj

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22

33
<PropertyGroup>
44
<Description>Serilog support for ASP.NET Core logging</Description>
5-
<VersionPrefix>4.1.1</VersionPrefix>
5+
<VersionPrefix>5.0.0</VersionPrefix>
66
<Authors>Microsoft;Serilog Contributors</Authors>
77
<TargetFrameworks>netstandard2.0;netstandard2.1;netcoreapp3.1;net5.0</TargetFrameworks>
88
<TreatWarningsAsErrors>true</TreatWarningsAsErrors>

0 commit comments

Comments
 (0)