Skip to content

Commit dd54e2f

Browse files
Remove opt-out - always use underscore in logging key.
1 parent fdfde80 commit dd54e2f

File tree

2 files changed

+6
-8
lines changed

2 files changed

+6
-8
lines changed

release-notes/wip-release-notes.md

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,8 @@ Date: ???
66

77
### Bugs
88

9+
- #111: Dots also get converted to underlines in structured keys.
10+
911
### Features
1012

1113
### Miscellaneous

src/Stravaig.Configuration.Diagnostics.Core/Renderers/Renderer.cs

Lines changed: 4 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -9,8 +9,6 @@ public abstract class Renderer
99
{
1010
private const string PlaceholderPartJoin = "_";
1111

12-
private bool _convertDotToUnderscore = false;
13-
1412
/// <summary>
1513
/// Creates a safely named placeholder for use in structured renderers.
1614
/// </summary>
@@ -34,12 +32,10 @@ protected string Placeholder(params string[] parts)
3432
{
3533
if (charPos == 0 && character >= '0' && character <= '9')
3634
placeholderBuilder.Append('_');
37-
if (char.IsLetterOrDigit(character))
38-
placeholderBuilder.Append(character);
39-
if (character == '.')
40-
placeholderBuilder.Append(_convertDotToUnderscore ? '_' : '.');
41-
else
42-
placeholderBuilder.Append('_');
35+
placeholderBuilder.Append(
36+
char.IsLetterOrDigit(character)
37+
? character
38+
: '_');
4339
charPos++;
4440
}
4541

0 commit comments

Comments
 (0)