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.

Sign up for GitHub

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

Jump to bottom

Save logevent as JSON #7

Merged
merged 6 commits into from
Dec 23, 2015
Merged

Save logevent as JSON #7

merged 6 commits into from
Dec 23, 2015

Conversation

rellis-of-rhindleton
Copy link
Contributor

  • Fixes a bug where properties with the same name (key) as a column would overwrite the column value.
  • Adds option to exclude properties from the Properties xml column if they are being saved as additional columns.
  • Adds option to save the log event JSON to new LogEvent nvarchar(max) column.

@mivano mivano merged commit 07132e3 into serilog-mssql:master Dec 23, 2015
@mivano
Copy link
Contributor

mivano commented Dec 23, 2015

Tested and changed the default value of store JSON to false so we do not break existing implementations. Thanks for the PR!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@rellis-of-rhindleton @mivano