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

Consider injecting EventId properties as normal message template parameters #540

Open
snakefoot opened this issue Oct 25, 2021 · 0 comments

Comments

@snakefoot
Copy link
Contributor

When receiving LogEvent that both contains message-template-parameters and EventId, then NLog will allocate both a message-template-parameter-array and also message-properties-dictionary (for eventid).

Maybe consider injecting EventId into the list of message-template-parameter, when message-template has been parsed as being "simple" (Not positional-placeholders and Not message-template-syntax and Not ParseMessageTemplates-option enabled).

@snakefoot snakefoot changed the title Consider adding EventId properties as normal message template parameters Consider injecting EventId properties as normal message template parameters Oct 26, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant