From 1d45a7606dbe53ffec422b48cb97ae1bb95ec4a3 Mon Sep 17 00:00:00 2001 From: Adam Stachowicz Date: Sun, 18 Jul 2021 13:51:44 +0200 Subject: [PATCH 1/2] Fix Docker build --- extra/mark-as-nightly.js | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/extra/mark-as-nightly.js b/extra/mark-as-nightly.js index c35455e87..28496511b 100644 --- a/extra/mark-as-nightly.js +++ b/extra/mark-as-nightly.js @@ -34,6 +34,7 @@ if (newVersion) { fs.writeFileSync("package.json", JSON.stringify(pkg, null, 4) + "\n") // Process README.md - fs.writeFileSync("README.md", fs.readFileSync("README.md", 'utf8').replaceAll(oldVersion, newVersion)) + if (fs.existsSync("README.md")) { + fs.writeFileSync("README.md", fs.readFileSync("README.md", 'utf8').replaceAll(oldVersion, newVersion)) + } } - From 9ca2444dabd4148fb695801291e5242733f0afb6 Mon Sep 17 00:00:00 2001 From: LouisLam Date: Sun, 18 Jul 2021 20:49:46 +0800 Subject: [PATCH 2/2] improve testing notification response --- server/notification.js | 96 +++++++++++++++++++++++++----------------- server/server.js | 9 +++- 2 files changed, 65 insertions(+), 40 deletions(-) diff --git a/server/notification.js b/server/notification.js index ff5298e6e..79c01f6db 100644 --- a/server/notification.js +++ b/server/notification.js @@ -5,12 +5,18 @@ const nodemailer = require("nodemailer"); const child_process = require("child_process"); class Notification { - static async send(notification, msg, monitorJSON = null, heartbeatJSON = null) { - let res = { - ok: true, - msg: "Sent Successfully" - } + /** + * + * @param notification + * @param msg + * @param monitorJSON + * @param heartbeatJSON + * @returns {Promise} Successful msg + * Throw Error with fail msg + */ + static async send(notification, msg, monitorJSON = null, heartbeatJSON = null) { + let okMsg = "Sent Successfully. "; if (notification.type === "telegram") { try { @@ -20,15 +26,16 @@ class Notification { text: msg, } }) - return true; + return okMsg; + } catch (error) { - console.error(error) - return false; + let msg = (error.response.data.description) ? error.response.data.description : "Error without description" + throw new Error(msg) } } else if (notification.type === "gotify") { try { - if (notification.gotifyserverurl.endsWith("/")) { + if (notification.gotifyserverurl && notification.gotifyserverurl.endsWith("/")) { notification.gotifyserverurl = notification.gotifyserverurl.slice(0, -1); } await axios.post(`${notification.gotifyserverurl}/message?token=${notification.gotifyapplicationToken}`, { @@ -36,15 +43,15 @@ class Notification { "priority": notification.gotifyPriority || 8, "title": "Uptime-Kuma" }) - return true; + + return okMsg; + } catch (error) { - console.error(error) - return false; + throwGeneralAxiosError(error) } } else if (notification.type === "webhook") { try { - let data = { heartbeat: heartbeatJSON, monitor: monitorJSON, @@ -66,10 +73,10 @@ class Notification { } let res = await axios.post(notification.webhookURL, finalData, config) - return true; + return okMsg; + } catch (error) { - console.error(error) - return false; + throwGeneralAxiosError(error) } } else if (notification.type === "smtp") { @@ -84,7 +91,7 @@ class Notification { content: msg } let res = await axios.post(notification.discordWebhookUrl, data) - return true; + return okMsg; } // If heartbeatJSON is not null, we go into the normal alerting loop. if(heartbeatJSON['status'] == 0) { @@ -110,10 +117,9 @@ class Notification { }] } let res = await axios.post(notification.discordWebhookUrl, data) - return true; + return okMsg; } catch(error) { - console.error(error) - return false; + throwGeneralAxiosError(error) } } else if (notification.type === "signal") { @@ -126,10 +132,9 @@ class Notification { let config = {}; let res = await axios.post(notification.signalURL, data, config) - return true; + return okMsg; } catch (error) { - console.error(error) - return false; + throwGeneralAxiosError(error) } } else if (notification.type === "slack") { @@ -137,7 +142,7 @@ class Notification { if (heartbeatJSON == null) { let data = {'text': "Uptime Kuma Slack testing successful.", 'channel': notification.slackchannel, 'username': notification.slackusername, 'icon_emoji': notification.slackiconemo} let res = await axios.post(notification.slackwebhookURL, data) - return true; + return okMsg; } const time = heartbeatJSON["time"]; @@ -182,10 +187,9 @@ class Notification { ] } let res = await axios.post(notification.slackwebhookURL, data) - return true; + return okMsg; } catch (error) { - console.error(error) - return false; + throwGeneralAxiosError(error) } } else if (notification.type === "pushover") { @@ -196,7 +200,7 @@ class Notification { 'user': notification.pushoveruserkey, 'token': notification.pushoverapptoken, 'sound':notification.pushoversounds, 'priority': notification.pushoverpriority, 'title':notification.pushovertitle, 'retry': "30", 'expire':"3600", 'html': 1} let res = await axios.post(pushoverlink, data) - return true; + return okMsg; } let data = { @@ -211,10 +215,9 @@ class Notification { "html": 1 } let res = await axios.post(pushoverlink, data) - return true; + return okMsg; } catch (error) { - console.log(error) - return false; + throwGeneralAxiosError(error) } } else if (notification.type === "apprise") { @@ -282,22 +285,24 @@ class Notification { text: msg, }); - return true; + return "Sent Successfully."; } static async apprise(notification, msg) { let s = child_process.spawnSync("apprise", [ "-vv", "-b", msg, notification.appriseURL]) - let output = s.stdout.toString(); - console.log(output) + + let output = (s.stdout) ? s.stdout.toString() : 'ERROR: maybe apprise not found'; if (output) { - return { - ok: ! output.includes("ERROR"), - msg: output + + if (! output.includes("ERROR")) { + return "Sent Successfully"; + } else { + throw new Error(output) } } else { - return { } + return "" } } @@ -306,6 +311,21 @@ class Notification { let exists = commandExistsSync('apprise'); return exists; } + +} + +function throwGeneralAxiosError(error) { + let msg = "Error: " + error + " "; + + if (error.response && error.response.data) { + if (typeof error.response.data === "string") { + msg += error.response.data; + } else { + msg += JSON.stringify(error.response.data) + } + } + + throw new Error(msg) } module.exports = { diff --git a/server/server.js b/server/server.js index ee27dc815..bd5894775 100644 --- a/server/server.js +++ b/server/server.js @@ -440,11 +440,16 @@ let needSetup = false; try { checkLogin(socket) - let res = await Notification.send(notification, notification.name + " Testing") + let msg = await Notification.send(notification, notification.name + " Testing") - callback(res); + callback({ + ok: true, + msg + }); } catch (e) { + console.error(e) + callback({ ok: false, msg: e.message