fix(notification-dingding): throw error when failed

[AnnAngela]

⚠️⚠️⚠️ Since we do not accept all types of pull requests and do not want to waste your time. Please be sure that you have read pull request rules:
https://github.com/louislam/uptime-kuma/blob/master/CONTRIBUTING.md#can-i-create-a-pull-request-for-uptime-kuma

Tick the checkbox if you understand [x]:

• I have read and understand the pull request rules.

Description

See #3573 (comment)

Type of change

Please delete any options that are not relevant.

• Bug fix (non-breaking change which fixes an issue)

Checklist

• My code follows the style guidelines of this project
• I ran ESLint and other linters for modified files
• I have performed a self-review of my own code and tested it
• I have commented my code, particularly in hard-to-understand areas
  (including JSDoc for methods)
• My changes generate no new warnings
• My code needed automated testing. I have added them (this is optional task)

Screenshots (if any)

#3598 (comment)

[CommanderStorm]

Have you tested that this works as intended?
Could you attach a screenshot as above of the monitor failing? (#3573 (comment)) ^^

[AnnAngela]

Have you tested that this works as intended?
Could you attach a screenshot as above of the monitor failing? (#3573 (comment)) ^^

OK, I will try to get the screenshot next Monday. (I'm on vacation.)

[CommanderStorm]

@AnnAngela ping ^^

[AnnAngela]

@CommanderStorm Sorry for late response, I forgot it :(

In this PR branch (wrong SecretKey used):

In the stable version (falsely success msg):

[CommanderStorm]

@AnnAngela Thanks a ton 🎉

@louislam FYI: I put this into the 1.23.12 milestone as I think this meets our requirements (only bugfixes) for said release and can be merged without more indepth reveiew as the "proof of working" we need is there ^^