Skip to content

Commit d266c60

Browse files
0xced0xced
committed
Add support for Serilog.Settings.Configuration
1 parent 3c3080d commit d266c60

9 files changed

+448
-10
lines changed

‎src/Log4NetTextFormatter.SerilogSettingsConfiguration.cs‎

Lines changed: 92 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,92 @@
1+
using System;
2+
using System.Diagnostics.CodeAnalysis;
3+
using System.Globalization;
4+
using System.Linq;
5+
6+
namespace Serilog.Formatting.Log4Net;
7+
8+
// Support for https://github.com/serilog/serilog-settings-configuration through dedicated constructor
9+
public partial class Log4NetTextFormatter
10+
{
11+
/// <summary>
12+
/// A highly improbable value to be used as the null text.
13+
/// </summary>
14+
private const string NullTextDefaultMarker = "$Serilog.Formatting.Log4Net.Log4NetTextFormatter.nullText$";
15+
16+
/// <summary>
17+
/// Do not use this constructor. It is only available for the <a href="https://github.com/serilog/serilog-settings-configuration">Serilog.Settings.Configuration</a> integration.
18+
/// The property filter, the message formatter and the exception formatter can only be configured through
19+
/// the <see cref="Log4NetTextFormatter(Action&lt;Log4NetTextFormatterOptionsBuilder&gt;)"/> constructor.
20+
/// </summary>
21+
/// <remarks>
22+
/// Binary compatibility across versions is not guaranteed. New optional parameters will be added in order to match new options.
23+
/// </remarks>
24+
[Obsolete("This constructor is only for use by the Serilog.Settings.Configuration package.", error: true)]
25+
[SuppressMessage("ReSharper", "UnusedMember.Global", Justification = "Used by Serilog.Settings.Configuration through reflection.")]
26+
public Log4NetTextFormatter(
27+
string? formatProvider = null,
28+
CDataMode? cDataMode = null,
29+
string? nullText = NullTextDefaultMarker,
30+
// in order to support options.UseNullText(null) on .NET < 10, see https://learn.microsoft.com/en-us/dotnet/core/compatibility/extensions/10.0/configuration-null-values-preserved
31+
bool noNullText = false,
32+
bool noXmlNamespace = false,
33+
LineEnding? lineEnding = null,
34+
string? indentation = null,
35+
bool log4JCompatibility = false
36+
) : this(options =>
37+
{
38+
if (formatProvider != null)
39+
options.UseFormatProvider(CultureInfo.GetCultureInfo(formatProvider));
40+
if (cDataMode != null)
41+
options.UseCDataMode(cDataMode.Value);
42+
if (nullText != NullTextDefaultMarker)
43+
options.UseNullText(nullText);
44+
if (noNullText)
45+
options.UseNullText(null);
46+
if (noXmlNamespace)
47+
options.UseNoXmlNamespace();
48+
if (lineEnding != null)
49+
options.UseLineEnding(lineEnding.Value);
50+
if (indentation != null)
51+
ConfigureIndentation(options, indentation);
52+
if (log4JCompatibility)
53+
options.UseLog4JCompatibility();
54+
})
55+
{
56+
}
57+
58+
private static void ConfigureIndentation(Log4NetTextFormatterOptionsBuilder options, string indentation)
59+
{
60+
if (indentation.Length == 0)
61+
{
62+
options.UseNoIndentation();
63+
return;
64+
}
65+
66+
var spaceCount = indentation.Count(c => c == ' ');
67+
if (spaceCount == indentation.Length)
68+
{
69+
if (spaceCount <= byte.MaxValue)
70+
{
71+
options.UseIndentationSettings(new IndentationSettings(Indentation.Space, Convert.ToByte(spaceCount)));
72+
return;
73+
}
74+
75+
throw new ArgumentException($"The indentation exceeds the maximum number of allowed spaces. ({spaceCount} > {byte.MaxValue})", nameof(indentation));
76+
}
77+
78+
var tabCount = indentation.Count(c => c == '\t');
79+
if (tabCount == indentation.Length)
80+
{
81+
if (tabCount <= byte.MaxValue)
82+
{
83+
options.UseIndentationSettings(new IndentationSettings(Indentation.Tab, Convert.ToByte(tabCount)));
84+
return;
85+
}
86+
87+
throw new ArgumentException($"The indentation exceeds the maximum number of allowed tabs. ({tabCount} > {byte.MaxValue})", nameof(indentation));
88+
}
89+
90+
throw new ArgumentException("The indentation must contains only space or tabs characters.", nameof(indentation));
91+
}
92+
}

‎src/Log4NetTextFormatter.cs‎

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,6 @@ namespace Serilog.Formatting.Log4Net;
1414
/// <summary>
1515
/// A text formatter that serialize log events into <a href="https://logging.apache.org/log4net/">log4net</a> or <a href="https://logging.apache.org/log4j/">log4j</a> compatible XML format.
1616
/// </summary>
17-
// ReSharper disable once PartialTypeWithSinglePart -- Used for [GeneratedRegex] on .NET 8 onwards
1817
public partial class Log4NetTextFormatter : ITextFormatter
1918
{
2019
/// <summary>

‎src/Log4NetTextFormatterOptions.cs‎

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -21,26 +21,26 @@ internal Log4NetTextFormatterOptions(IFormatProvider? formatProvider, CDataMode
2121
}
2222

2323
/// <summary>See <see cref="Log4NetTextFormatterOptionsBuilder.UseFormatProvider"/></summary>
24-
internal IFormatProvider? FormatProvider { get; }
24+
public IFormatProvider? FormatProvider { get; }
2525

2626
/// <summary>See <see cref="Log4NetTextFormatterOptionsBuilder.UseCDataMode"/></summary>
27-
internal CDataMode CDataMode { get; }
27+
public CDataMode CDataMode { get; }
2828

2929
/// <summary>See <see cref="Log4NetTextFormatterOptionsBuilder.UseNullText"/></summary>
30-
internal string? NullText { get; }
30+
public string? NullText { get; }
3131

3232
/// <summary>See <see cref="Log4NetTextFormatterOptionsBuilder.UseNoXmlNamespace"/></summary>
33-
internal XmlQualifiedName? XmlNamespace { get; }
33+
public XmlQualifiedName? XmlNamespace { get; }
3434

3535
/// <summary>See <see cref="Log4NetTextFormatterOptionsBuilder.CreateXmlWriterSettings"/></summary>
36-
internal XmlWriterSettings XmlWriterSettings { get; }
36+
public XmlWriterSettings XmlWriterSettings { get; }
3737

3838
/// <summary>See <see cref="Log4NetTextFormatterOptionsBuilder.UsePropertyFilter"/></summary>
39-
internal PropertyFilter FilterProperty { get; }
39+
public PropertyFilter FilterProperty { get; }
4040

4141
/// <summary>See <see cref="Log4NetTextFormatterOptionsBuilder.UseMessageFormatter"/></summary>
42-
internal MessageFormatter FormatMessage { get; }
42+
public MessageFormatter FormatMessage { get; }
4343

4444
/// <summary>See <see cref="Log4NetTextFormatterOptionsBuilder.UseExceptionFormatter"/></summary>
45-
internal ExceptionFormatter FormatException { get; }
45+
public ExceptionFormatter FormatException { get; }
4646
}

‎src/Log4NetTextFormatterOptionsBuilder.cs‎

Lines changed: 11 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,16 @@ public class Log4NetTextFormatterOptionsBuilder
2121
/// <remarks>https://github.com/apache/log4j/blob/v1_2_17/src/main/java/org/apache/log4j/xml/XMLLayout.java#L137</remarks>
2222
internal static readonly XmlQualifiedName Log4JXmlNamespace = new("log4j", "http://jakarta.apache.org/log4j/");
2323

24+
/// <summary>
25+
/// The default indentation style, i.e. <c>space</c>
26+
/// </summary>
27+
internal const Indentation DefaultIndentation = Indentation.Space;
28+
29+
/// <summary>
30+
/// The default indentation size, i.e. <c>2</c>
31+
/// </summary>
32+
internal const byte DefaultIndentationSize = 2;
33+
2434
/// <summary>
2535
/// Initialize a new instance of the <see cref="Log4NetTextFormatterOptionsBuilder"/> class.
2636
/// </summary>
@@ -44,7 +54,7 @@ internal Log4NetTextFormatterOptionsBuilder()
4454
private LineEnding _lineEnding = LineEnding.LineFeed;
4555

4656
/// <summary>See <see cref="UseIndentationSettings"/></summary>
47-
private IndentationSettings? _indentationSettings = new(Indentation.Space, size: 2);
57+
private IndentationSettings? _indentationSettings = new(DefaultIndentation, DefaultIndentationSize);
4858

4959
/// <summary>See <see cref="UsePropertyFilter"/></summary>
5060
private PropertyFilter _filterProperty = (_, _) => true;

‎src/Serilog.Formatting.Log4Net.csproj‎

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -57,6 +57,10 @@
5757
<None Update="packages.lock.json" Visible="false" />
5858
</ItemGroup>
5959

60+
<ItemGroup Label="Testing">
61+
<InternalsVisibleTo Include="Serilog.Formatting.Log4Net.Tests" />
62+
</ItemGroup>
63+
6064
<ItemGroup>
6165
<PackageReference Include="MinVer" Version="6.0.0" PrivateAssets="all" />
6266
<PackageReference Include="Serilog" Version="2.0.0" />

‎tests/PublicApi.net8.0.verified.cs‎

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
[assembly: System.CLSCompliant(true)]
22
[assembly: System.Reflection.AssemblyMetadata("IsTrimmable", "True")]
33
[assembly: System.Reflection.AssemblyMetadata("RepositoryUrl", "https://github.com/serilog-contrib/serilog-formatting-log4net")]
4+
[assembly: System.Runtime.CompilerServices.InternalsVisibleTo("Serilog.Formatting.Log4Net.Tests")]
45
[assembly: System.Runtime.Versioning.TargetFramework(".NETCoreApp,Version=v8.0", FrameworkDisplayName=".NET 8.0")]
56
namespace Serilog.Formatting.Log4Net
67
{
@@ -32,6 +33,8 @@ public class Log4NetTextFormatter : Serilog.Formatting.ITextFormatter
3233
{
3334
public Log4NetTextFormatter() { }
3435
public Log4NetTextFormatter(System.Action<Serilog.Formatting.Log4Net.Log4NetTextFormatterOptionsBuilder>? configureOptions) { }
36+
[System.Obsolete("This constructor is only for use by the Serilog.Settings.Configuration package.", true)]
37+
public Log4NetTextFormatter(string? formatProvider = null, Serilog.Formatting.Log4Net.CDataMode? cDataMode = default, string? nullText = "$Serilog.Formatting.Log4Net.Log4NetTextFormatter.nullText$", bool noNullText = false, bool noXmlNamespace = false, Serilog.Formatting.Log4Net.LineEnding? lineEnding = default, string? indentation = null, bool log4JCompatibility = false) { }
3538
public static Serilog.Formatting.Log4Net.Log4NetTextFormatter Log4JFormatter { get; }
3639
public void Format(Serilog.Events.LogEvent logEvent, System.IO.TextWriter output) { }
3740
}

‎tests/PublicApi.netstandard2.0.verified.cs‎

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
[assembly: System.CLSCompliant(true)]
22
[assembly: System.Reflection.AssemblyMetadata("RepositoryUrl", "https://github.com/serilog-contrib/serilog-formatting-log4net")]
3+
[assembly: System.Runtime.CompilerServices.InternalsVisibleTo("Serilog.Formatting.Log4Net.Tests")]
34
[assembly: System.Runtime.Versioning.TargetFramework(".NETStandard,Version=v2.0", FrameworkDisplayName=".NET Standard 2.0")]
45
namespace Serilog.Formatting.Log4Net
56
{
@@ -31,6 +32,8 @@ public class Log4NetTextFormatter : Serilog.Formatting.ITextFormatter
3132
{
3233
public Log4NetTextFormatter() { }
3334
public Log4NetTextFormatter(System.Action<Serilog.Formatting.Log4Net.Log4NetTextFormatterOptionsBuilder>? configureOptions) { }
35+
[System.Obsolete("This constructor is only for use by the Serilog.Settings.Configuration package.", true)]
36+
public Log4NetTextFormatter(string? formatProvider = null, Serilog.Formatting.Log4Net.CDataMode? cDataMode = default, string? nullText = "$Serilog.Formatting.Log4Net.Log4NetTextFormatter.nullText$", bool noNullText = false, bool noXmlNamespace = false, Serilog.Formatting.Log4Net.LineEnding? lineEnding = default, string? indentation = null, bool log4JCompatibility = false) { }
3437
public static Serilog.Formatting.Log4Net.Log4NetTextFormatter Log4JFormatter { get; }
3538
public void Format(Serilog.Events.LogEvent logEvent, System.IO.TextWriter output) { }
3639
}

‎tests/Serilog.Formatting.Log4Net.Tests.csproj‎

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,16 +8,23 @@
88
<ItemGroup>
99
<PackageReference Include="AwesomeAssertions" Version="9.1.0" />
1010
<PackageReference Include="coverlet.collector" Version="6.0.4" PrivateAssets="all" />
11+
<PackageReference Include="Microsoft.Extensions.Configuration.Json" Version="9.0.9" />
1112
<PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.14.1" />
13+
<PackageReference Include="NuGet.Versioning" Version="6.14.0" />
1214
<PackageReference Include="PublicApiGenerator" Version="11.4.6" />
15+
<PackageReference Include="ReflectionMagic" Version="5.0.1" />
1316
<PackageReference Include="ReportGenerator" Version="5.4.14" PrivateAssets="all" />
1417
<PackageReference Include="Serilog" Version="4.3.0" />
1518
<PackageReference Include="Serilog.Enrichers.Environment" Version="3.0.1" />
1619
<PackageReference Include="Serilog.Enrichers.Thread" Version="4.0.0" />
20+
<PackageReference Include="Serilog.Settings.Configuration" Version="9.0.0" />
21+
<PackageReference Include="Serilog.Sinks.File" Version="7.0.0" />
22+
<PackageReference Include="Tomlyn.Extensions.Configuration" Version="1.0.6" />
1723
<PackageReference Include="Verify" Version="30.18.0" GeneratePathProperty="true" />
1824
<PackageReference Include="Verify.Xunit" Version="30.18.0" />
1925
<PackageReference Include="xunit" Version="2.9.3" />
2026
<PackageReference Include="xunit.runner.visualstudio" Version="3.1.4" PrivateAssets="all" />
27+
<PackageReference Include="Xunit.SkippableFact" Version="1.5.23" />
2128
</ItemGroup>
2229

2330
<ItemGroup>

0 commit comments

Comments
 (0)