search

serilog / serilog-extensions-logging

Serilog provider for Microsoft.Extensions.Logging
Apache License 2.0
307 stars 97 forks source link

added stringification support #160

Closed jonnybi closed 4 years ago

jonnybi commented 4 years ago

Added "force stringification" support as described in serilog documentation.

nblumhardt commented 4 years ago

@jonnybi hi! I just spotted one more place where we special-case @, which I think should also accept $ for the sake of consistency:

https://github.com/serilog/serilog-extensions-logging/blob/dev/src/Serilog.Extensions.Logging/Extensions/Logging/SerilogLoggerScope.cs#L75

jonnybi commented 4 years ago

@nblumhardt thx for spotting this. i will check it

nblumhardt commented 4 years ago

Thanks! 👍