Telegram notification support

[islander]

Implement sending notification to Telegram chats

[jamtur01]

Looks like some test failures in there.

[pradeepchhetri]

You need to add clojure.string/escape here, that's why the test is failing.

[pradeepchhetri-nd]

@islander Also you need to update these files as well:

https://github.com/riemann/riemann/blob/master/project.clj
https://github.com/riemann/riemann/blob/master/src/riemann/config.clj

[islander]

@pradeepchhetri-nd thank, i'll take a look later.

So, I just need to add [telegram :refer [telegram]] in :require section of riemann/config.clj and :telegram :telegram to :test-selectors of project.clj?

P.S. Sorry for my mistakes and stupid questions, I'm new to clojure and riemann development =)

[jamtur01]

@islander Yes.

[jamtur01]

LGTM - excluding the configurable formatting.

[pradeepchhetri-nd]

@jamtur01 @islander I will try this on my setup today and let you know how the testing goes. Thanks.

[jamtur01]

@pradeepchhetri I am not overly keen on the sleep....

[pradeepchhetri]

@jamtur01 @islander I think we can do something better than sleep Will get back to you soon.

[islander]

@jamtur01 by external formating you mean switching between HTML and Markdown? I'll try to add this soon. And what other options we have to replace sleep?

[islander]

According to formatting: sometimes I send, for example, syslog message in event description, so telegram markdown parser could interpret all special symbols in message ([,],*,_ and so on) and formatting become weird. So, should I make escaping inside format-message function, or it should be done outside of it (in client, or somewhere in riemann streams)? It's the reason i switched to HTML parse_mode (easier to escape :))

example of such message:

Description: WARNING[817][C-00036738]: app_dial.c:2437 in dial_exec_full: Unable to create channel of type 'SIP' (cause 20 - Subscriber absent)

becomes something like this:

Description: WARNING817C-00036738: app dial.c:2437 in dial exec_full: Unable to create channel of type 'SIP' (cause 20 - Subscriber absent)

[pradeepchhetri]

@islander I send a pull request to your fork's telegram branch. I tested the plugin and works perfectly fine.

[islander]

I've added parse_mode option.

[pyr]

thanks @islander