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

XmlLayout - Fixed missing encode of xml element value #3198

Merged
merged 1 commit into from
Mar 25, 2019

Conversation

snakefoot
Copy link
Contributor

@snakefoot snakefoot commented Mar 25, 2019

Bug introduced with the last major refactoring before NLog 4.6

@codecov
Copy link

codecov bot commented Mar 25, 2019

Codecov Report

Merging #3198 into dev will decrease coverage by <1%.
The diff coverage is 50%.

@@          Coverage Diff           @@
##             dev   #3198    +/-   ##
======================================
- Coverage     80%     80%   -<1%     
======================================
  Files        354     354            
  Lines      27909   27913     +4     
  Branches    3706    3706            
======================================
+ Hits       22241   22242     +1     
- Misses      4599    4608     +9     
+ Partials    1069    1063     -6

@304NotModified 304NotModified added this to the 4.6.1 milestone Mar 25, 2019
@304NotModified 304NotModified added bug Bug report / Bug fix xml-layout labels Mar 25, 2019
@304NotModified 304NotModified merged commit 85f876f into NLog:dev Mar 25, 2019
@304NotModified
Copy link
Member

Thanks!

@snakefoot snakefoot deleted the XmlLayoutMissingEncode branch April 4, 2020 12:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Bug report / Bug fix xml-layout
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants