Add Watchdog verbose logging

[MAGICCC]

Adding the possibility to see verbose logs while sending mails via Watchdog
May fix #4298 + #4197

PS: Needs a new watchdog image then
PPS: Needs more testing - only tested it quickly. @andryyy maybe you have some time for this

[andryyy]

Looks good in my test, but why? Did you find an issue regarding this? Is it normal?

[MAGICCC]

Looks like it. I found mludvig/smtp-cli#17 and seems exitcode is 1 when everything is fine...

[andryyy]

Nice. :) Let me know when you can merge. I will build the image then first.

[MAGICCC]

I just tested a bit and looks fine on my side, but I noticed when you set a mail with a bad domain (no mx record), then no mail will be sent.
Maybe we can reuse this:

mailcow-dockerized/data/Dockerfiles/watchdog/watchdog.sh

Lines 129 to 135 in 8b4e1ce

	RCPT_DOMAIN=$(echo ${rcpt} | awk -F @ {'print $NF'})
	# Latest smtp-cli looks up mx via dns
	#RCPT_MX=$(dig +short ${RCPT_DOMAIN} mx | sort -n | awk '{print $2; exit}')
	#if [[ -z ${RCPT_MX} ]]; then
	# log_msg "Cannot determine MX for ${rcpt}, skipping email notification..."
	# return 1
	#fi

like this:

RCPT_DOMAIN=$(echo ${rcpt} | awk -F @ {'print $NF'})
CHECK_FOR_VALID_MX=$(dig +short ${RCPT_DOMAIN} mx | sort -n | awk '{print $2; exit}')
if [[ -z ${CHECK_FOR_VALID_MX} ]]; then
  log_msg "Cannot determine MX for ${rcpt}, skipping email notification..."
  return 1
fi

[andryyy]

CHECK_FOR_VALID_MX=$(dig +short ${RCPT_DOMAIN} mx) would be enough. :)

Can you adjust the PR?

[andryyy]

Thank youuu

[andryyy]

We should probably add the variable information to the docs.

[dragoangel]

Wow, nice :)