fix_irc_notification: Remove line break from message #7243
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
Fixes #7233
When Netdata is sending messages to IRC channels, the messages are generating the error 421, because our default message is not according RFC1459(https://tools.ietf.org/html/rfc1459#section-2.3.1 ), this PR replaces the line break with commas to fix the problem.
Component Name
Alarms
Additional Information
I used the server ircd-hybrid to develop, because I wanted to have messages from both sides, but it is possible to do a simpler test, this PR must avoid the error.log to have error messages 421 when we send an alarm.