Skip to content

JSON encoding should create valid JSON for special double values (hotfix)#2865

Merged
304NotModified merged 2 commits intoNLog:masterfrom
snakefoot:JsonInvalidDoubleEncode2
Aug 24, 2018
Merged

JSON encoding should create valid JSON for special double values (hotfix)#2865
304NotModified merged 2 commits intoNLog:masterfrom
snakefoot:JsonInvalidDoubleEncode2

Conversation

@snakefoot
Copy link
Copy Markdown
Contributor

Resolves #2860 by handling NaN and Infinity (Also for floats)

@codecov
Copy link
Copy Markdown

codecov bot commented Aug 23, 2018

Codecov Report

Merging #2865 into master will increase coverage by <1%.
The diff coverage is 87%.

@@           Coverage Diff            @@
##           master   #2865     +/-   ##
========================================
+ Coverage      81%     81%    +<1%     
========================================
  Files         326     327      +1     
  Lines       24414   25464   +1050     
  Branches     3107    3415    +308     
========================================
+ Hits        19697   20628    +931     
- Misses       3851    3952    +101     
- Partials      866     884     +18

@304NotModified 304NotModified changed the title JSON encoding should create valid JSON for special double values JSON encoding should create valid JSON for special double values (hotfix) Aug 24, 2018
@304NotModified 304NotModified merged commit 1d1ef42 into NLog:master Aug 24, 2018
@304NotModified
Copy link
Copy Markdown
Member

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Bug report / Bug fix json / json-layout

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants