Repository: louislam/uptime-kuma
File: server/notification-providers/heii-oncall.js (JavaScript)

Code before:
const { UP, DOWN, getMonitorRelativeURL } = require("../../src/util");
const { setting } = require("../util-server");

const NotificationProvider = require("./notification-provider");
const axios = require("axios");

const heiiOnCallBaseUrl = "https://heiioncall.com";

class HeiiOnCall extends NotificationProvider {
    name = "HeiiOnCall";

    /**
     * @inheritdoc
     */
    async send(notification, msg, monitorJSON = null, heartbeatJSON = null) {
        // Payload to Heii On-Call is the entire heartbat JSON
        const payload = heartbeatJSON ? heartbeatJSON : {};

        // If we can, add url back to monitor to payload
        const baseURL = await setting("primaryBaseURL");
        if (baseURL && monitorJSON) {
            payload["url"] = baseURL + getMonitorRelativeURL(monitorJSON.id);
        }

        if (!heartbeatJSON) {
            // No heartbeatJSON. Could be test button, but not necessarily. Just pull msg into payload.
            payload["msg"] = msg;
            return this.postNotification(notification, "alert", payload);
        }

        if (heartbeatJSON.status === DOWN) {
            // Monitor is DOWN, alert on Heii On-Call
            return this.postNotification(notification, "alert", payload);
        }

        if (heartbeatJSON.status === UP) {
            // Monitor is UP, resolve on Heii On-Call
            return this.postNotification(notification, "resolve", payload);
        }
    }

    /**
     * Post to Heii On-Call
     * @param {BeanModel} notification Message title
     * @param {string} action Trigger action (alert, resovle)
     * @param {object} payload Data for Heii On-Call
     * @returns {Promise<string>} Success message
     */
    async postNotification(notification, action, payload) {
        const config = {

Review comment (suggestion):
(not a suggestion, because this should be suggested from an IDE, not from github)

Please wrap these blocks in a big `try-catch-throwGeneralAxiosError` from below and inline the other functions.

The rationale behind this request is making the controflow more obvious (i.e. that we always return `Sent Successfully` for the successful case)

Code after:
const { UP, DOWN, getMonitorRelativeURL } = require("../../src/util");
const { setting } = require("../util-server");

const NotificationProvider = require("./notification-provider");
const axios = require("axios");
class HeiiOnCall extends NotificationProvider {
    name = "HeiiOnCall";

    /**
     * @inheritdoc
     */
    async send(notification, msg, monitorJSON = null, heartbeatJSON = null) {
        const okMsg = "Sent Successfully.";
        const payload = heartbeatJSON || {};

        const baseURL = await setting("primaryBaseURL");
        if (baseURL && monitorJSON) {
            payload["url"] = baseURL + getMonitorRelativeURL(monitorJSON.id);
        }

        const config = {
            headers: {
                Accept: "application/json",
                "Content-Type": "application/json",
                Authorization: "Bearer " + notification.heiiOnCallApiKey,
            },
        };
        const heiiUrl = `https://heiioncall.com/triggers/${notification.heiiOnCallTriggerId}/`;
        // docs https://heiioncall.com/docs#manual-triggers
        try {
            if (!heartbeatJSON) {
                // Testing or general notification like certificate expiry
                payload["msg"] = msg;
                await axios.post(heiiUrl + "alert", payload, config);
                return okMsg;
            }

            if (heartbeatJSON.status === DOWN) {
                await axios.post(heiiUrl + "alert", payload, config);
                return okMsg;
            }
            if (heartbeatJSON.status === UP) {
                await axios.post(heiiUrl + "resolve", payload, config);
                return okMsg;
            }
        } catch (error) {
            this.throwGeneralAxiosError(error);
        }
    }
}