Skip to content
This repository has been archived by the owner on Aug 8, 2023. It is now read-only.

Fixed new-lines handling #16

Closed
wants to merge 1 commit into from
Closed

Conversation

tarabanton
Copy link

When using this script with Rocket.Chat, it seems that server doesn't handle newlines properly.
Added new line substitution to \\n
Please test with Slack and Mattermost if possible.

When using this script with Rocket.Chat, it seems that server doesn't handle newlines properly.
Added new line substitution to `\\n`
Please test with Slack and Mattermost if possible.
@neo-inuit
Copy link

neo-inuit commented Nov 16, 2016

Hello tarabanton,

Thanks for the tip,
We have to use the text="${3//$'\r'/\\r}" to work with mattermost

@ericoc
Copy link
Owner

ericoc commented Dec 22, 2016

I'm not going to merge these changes in since I don't really intend for this to be used on Mattermost nor do I have a place to test it with Mattermost, but I'm going to reference this PR in the README.

@ericoc ericoc closed this Dec 22, 2016
ericoc added a commit that referenced this pull request Dec 22, 2016
sander85 added a commit to sander85/zabbix-mattermost-alertscript that referenced this pull request Sep 6, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants