Feat: Write full message into short_message field#24
Feat: Write full message into short_message field#24valentin-krasontovitsch wants to merge 1 commit intoGraylog2:v2from
Conversation
Following the principle of least surprise, we write messages completely into thethe `short_message` field, even if they contain several lines. As a consequence, we deprecate the `full_message` field, as it becomes redundant. Resolves Graylog2#11
|
Since this is an API change (functionality of If you accept the merge request for more precise time stamps first, I'll be happy to rebase this guy on top of that. |
|
Thanks for adding this. What if we can consider this PR a patch (like, for a bug fix), and let the merge in v2? 😄 |
|
@valentin-krasontovitsch thanks again for this PR! I would not simply remove the |
|
Okay finally coming around to looking at this again - sorry for the incredible delay... Closing in favor of #34, which writes the whole message into both fields, period - hope this is what' desired ^^ |
Following the principle of least surprise, we write messages completely into
thethe
short_messagefield, even if they contain several lines.As a consequence, we deprecate the
full_messagefield, as it becomesredundant.
Resolves #11