Merge pull request #19 from Elompenta/curl-timeout-breaking-all-other-notifies

Fix "no overall checkmk notifications anymore" if curl timeout happens

Author: filipnet

README.md

@@ -186,7 +186,7 @@ Edit your user and enter your Group/Chat-ID
 
 ## TROUBLESHOOTING
 For more details and troubleshooting with parameters please check:
-* Check_MK notification logfile: /omd/sites/{sitename}/var/log/notify.log
+* Check_MK notification logfile: ```tail -f /omd/sites/{sitename}/var/log/notify.log```
 * [Check_MK  Manual > Notifications > Chapter: 11.3. A simple example](https://docs.checkmk.com/latest/en/notifications.html#H1:Real)
 * [[Feature-Request] Multiple Alert Profiles](https://github.com/filipnet/checkmk-telegram-notify/issues/3)
 

check_mk_telegram-notify.sh

@@ -102,11 +102,12 @@ MESSAGE+="%0AIPv4: ${NOTIFY_HOST_ADDRESS_4} %0AIPv6: ${NOTIFY_HOST_ADDRESS_6}%0A
 MESSAGE+="${NOTIFY_SHORTDATETIME} | ${OMD_SITE}"
 
 # Send message to Telegram bot
-response=$(curl -S -s -q -X POST "https://api.telegram.org/bot${TOKEN}/sendMessage" -d chat_id="${CHAT_ID}" -d text="${MESSAGE}")
+response=$(curl -S -s -q --connect-timeout 5 -X POST "https://api.telegram.org/bot${TOKEN}/sendMessage" -d chat_id="${CHAT_ID}" -d text="${MESSAGE}")
 if [ $? -ne 0 ]; then
         echo "Not able to send Telegram message" >&2
         echo $response >&2
         exit 2
 else
+        echo "Telegramm message send sucessfull" >&2
         exit 0
 fi