Update Properties names and values.

serilog-contrib / Serilog.Sinks.Logz.Io

Apache License 2.0

15 stars 11 forks source link

Update Properties names and values. #3

Closed rafaelsc closed 6 years ago

rafaelsc commented 6 years ago

Adds Tests
Use lowercase Properties names. (This is based on the documentation and Loj4J Adaptor from LogzIo)
Use HTTPS by default (Fix #1)
Serialization for the Properties values. (Fix #2)
Update Readme.MD.

This Update can be a breaking change. For that reason, I recommend update to version 2 after this update.

brazabr commented 6 years ago

I work with @rafaelsc and want to clarify why this change breaks compatibility. If there is already an index in Logz.io with a field mapped to some type and the type being sent by the new serializer is invalid, the log entry is discarded. This may be a huge deal breaker for people running the existing code.