Fix

.github/ISSUE_TEMPLATE/ask_for_help.yml

@@ -3,7 +3,7 @@ name: ❓ Ask for help
 description: |
   Submit any question related to Uptime Kuma
 #title: "[Help]"
-labels: ["help"]
+labels: ["help", "P3-low"]
 body:
   - type: markdown
     attributes:

.github/ISSUE_TEMPLATE/bug_report.yml

@@ -3,7 +3,7 @@ name: 🐛 Bug Report
 description: |
   Submit a bug report to help us improve
 #title: "[Bug]"
-labels: ["bug"]
+labels: ["bug", "P2-medium"]
 body:
   - type: markdown
     attributes:

.github/ISSUE_TEMPLATE/feature_request.yml

@@ -3,7 +3,7 @@ name: 🚀 Feature Request
 description: |
   Submit a proposal for a new feature
 # title: "[Feature]"
-labels: ["feature-request"]
+labels: ["feature-request", "P3-low"]
 body:
   - type: markdown
     attributes:

.github/ISSUE_TEMPLATE/security_issue.yml

@@ -3,7 +3,7 @@ name: 🛡️ Security Issue
 description: |
   Notify Louis Lam about a security concern. Please do NOT include any sensitive details in this issue.
 # title: "Security Issue"
-labels: ["security"]
+labels: ["security", "P1-high"]
 assignees: [louislam]
 body:
   - type: markdown

.github/PULL_REQUEST_TEMPLATE.md

@@ -1,10 +1,10 @@
+**⚠️ Please Note: We do not accept all types of pull requests, and we want to ensure we don’t waste your time. Before submitting, make sure you have read our pull request guidelines: [Pull Request Rules](https://github.com/louislam/uptime-kuma/blob/master/CONTRIBUTING.md#can-i-create-a-pull-request-for-uptime-kuma)**
+
 ## ❗ Important Announcement
 
 <details><summary>Click here for more details:</summary>
 </p>
 
-**⚠️ Please Note: We do not accept all types of pull requests, and we want to ensure we don’t waste your time. Before submitting, make sure you have read our pull request guidelines: [Pull Request Rules](https://github.com/louislam/uptime-kuma/blob/master/CONTRIBUTING.md#can-i-create-a-pull-request-for-uptime-kuma)**
-
 ### 🚧 Temporary Delay in Feature Requests and Pull Request Reviews
 
 **At this time, we may be slower to respond to new feature requests and review pull requests. Existing requests and PRs will remain in the backlog but may not be prioritized immediately.**
@@ -26,22 +26,16 @@ We appreciate your patience and understanding as we continue to improve Uptime K
 
 ## 📋 Overview
 
-<!-- Provide a clear summary of the purpose and scope of this pull request:-->
+Provide a clear summary of the purpose and scope of this pull request:
 
 - **What problem does this pull request address?**
+
   - Please provide a detailed explanation here.
+
 - **What features or functionality does this pull request introduce or enhance?**
+
   - Please provide a detailed explanation here.
 
-## 🔗 Related Issues
-
-<!--
-Please link any GitHub issues or tasks that this pull request addresses. Use the appropriate issue numbers or links.
--->
-
-- Relates to #issue-number
-- Resolves #issue-number
-
 ## 🔄 Changes
 
 ### 🛠️ Type of change
@@ -58,7 +52,19 @@ Please link any GitHub issues or tasks that this pull request addresses. Use the
 - [ ] 🔧 Other (please specify):
   - Provide additional details here.
 
-## 📄 Checklist
+## 🔗 Related Issues
+
+<!--
+Please link any GitHub issues or tasks that this pull request addresses. Use the appropriate issue numbers or links.
+
+**Note**: Include only issues directly related to this PR. Remove any irrelevant reference.
+-->
+
+- Relates to #issue-number
+- Resolves #issue-number
+- Fixes #issue-number
+
+## 📄 Checklist *
 
 <!-- Please select all options that apply -->
 
@@ -91,3 +97,26 @@ If not, remove this section.
 | `DOWN`             | ![Before](image-link) | ![After](image-link) |
 | Certificate-expiry | ![Before](image-link) | ![After](image-link) |
 | Testing            | ![Before](image-link) | ![After](image-link) |
+
+## ℹ️ Additional Context
+
+Provide any relevant details to assist reviewers in understanding the changes.
+
+<details><summary>Click here for more details:</summary>
+</p>
+
+**Key Considerations**:
+
+- **Design decisions** – Key choices or trade-offs made during development.
+- **Alternative solutions** – Approaches considered but not implemented, along with reasons.
+- **Relevant links** – Specifications, discussions, or resources that provide context.
+- **Dependencies** – Related pull requests or issues that must be resolved before merging.
+- **Additional context** – Any other details that may help reviewers understand the changes.
+
+Provide details here
+
+## 💬 Requested Feedback
+
+<!-- If a part of our docs is unclear, you are unsure how to do something/.. this is where we would appreciate your feedback -->
+
+- `Mention documents needing feedback here`

docker/dockerfile

@@ -79,10 +79,6 @@ USER node
 RUN git config --global user.email "no-reply@no-reply.com"
 RUN git config --global user.name "PR Tester"
 RUN git clone https://github.com/louislam/uptime-kuma.git .
-
-# Hide the warning when running in detached head state
-RUN git config --global advice.detachedHead false
-
 RUN npm ci
 
 EXPOSE 3000 3001

extra/checkout-pr.js

@@ -0,0 +1,33 @@
+const childProcess = require("child_process");
+
+if (!process.env.UPTIME_KUMA_GH_REPO) {
+    console.error("Please set a repo to the environment variable 'UPTIME_KUMA_GH_REPO' (e.g. mhkarimi1383:goalert-notification)");
+    process.exit(1);
+}
+
+let inputArray = process.env.UPTIME_KUMA_GH_REPO.split(":");
+
+if (inputArray.length !== 2) {
+    console.error("Invalid format. Please set a repo to the environment variable 'UPTIME_KUMA_GH_REPO' (e.g. mhkarimi1383:goalert-notification)");
+}
+
+let name = inputArray[0];
+let branch = inputArray[1];
+
+console.log("Checkout pr");
+
+// Checkout the pr
+let result = childProcess.spawnSync("git", [ "remote", "add", name, `https://github.com/${name}/uptime-kuma` ]);
+
+console.log(result.stdout.toString());
+console.error(result.stderr.toString());
+
+result = childProcess.spawnSync("git", [ "fetch", name, branch ]);
+
+console.log(result.stdout.toString());
+console.error(result.stderr.toString());
+
+result = childProcess.spawnSync("git", [ "checkout", `${name}/${branch}`, "--force" ]);
+
+console.log(result.stdout.toString());
+console.error(result.stderr.toString());

extra/checkout-pr.mjs

@@ -1,34 +0,0 @@
-import childProcess from "child_process";
-import { parsePrName } from "./kuma-pr/pr-lib.mjs";
-
-let { name, branch } = parsePrName(process.env.UPTIME_KUMA_GH_REPO);
-
-console.log(`Checking out PR from ${name}:${branch}`);
-
-// Checkout the pr
-let result = childProcess.spawnSync("git", [ "remote", "add", name, `https://github.com/${name}/uptime-kuma` ], {
-    stdio: "inherit"
-});
-
-if (result.status !== 0) {
-    console.error("Failed to add remote repository.");
-    process.exit(1);
-}
-
-result = childProcess.spawnSync("git", [ "fetch", name, branch ], {
-    stdio: "inherit"
-});
-
-if (result.status !== 0) {
-    console.error("Failed to fetch the branch.");
-    process.exit(1);
-}
-
-result = childProcess.spawnSync("git", [ "checkout", `${name}/${branch}`, "--force" ], {
-    stdio: "inherit"
-});
-
-if (result.status !== 0) {
-    console.error("Failed to checkout the branch.");
-    process.exit(1);
-}

extra/kuma-pr/index.mjs

@@ -1,26 +0,0 @@
-#!/usr/bin/env node
-import { spawn } from "child_process";
-import { parsePrName } from "./pr-lib.mjs";
-
-const prName = process.argv[2];
-
-// Pre-check the prName here, so testers don't need to wait until the Docker image is pulled to see the error.
-try {
-    parsePrName(prName);
-} catch (error) {
-    console.error(error.message);
-    process.exit(1);
-}
-
-spawn("docker", [
-    "run",
-    "--rm",
-    "-it",
-    "-p", "3000:3000",
-    "-p", "3001:3001",
-    "--pull", "always",
-    "-e", `UPTIME_KUMA_GH_REPO=${prName}`,
-    "louislam/uptime-kuma:pr-test2"
-], {
-    stdio: "inherit",
-});

extra/kuma-pr/package.json

@@ -1,8 +0,0 @@
-{
-    "name": "kuma-pr",
-    "version": "1.0.0",
-    "type": "module",
-    "bin": {
-        "kuma-pr": "./index.mjs"
-    }
-}

extra/kuma-pr/pr-lib.mjs

@@ -1,39 +0,0 @@
-/**
- * Parse <name>:<branch> to an object.
- * @param {string} prName <name>:<branch>
- * @returns {object} An object with name and branch properties.
- */
-export function parsePrName(prName) {
-    let name = "louislam";
-    let branch;
-
-    const errorMessage = "Please set a repo to the environment variable 'UPTIME_KUMA_GH_REPO' (e.g. mhkarimi1383:goalert-notification)";
-
-    if (!prName) {
-        throw new Error(errorMessage);
-    }
-
-    prName = prName.trim();
-    if (prName === "") {
-        throw new Error(errorMessage);
-    }
-
-    let inputArray = prName.split(":");
-
-    // Just realized that owner's prs are not prefixed with "louislam:"
-    if (inputArray.length === 1) {
-        branch = inputArray[0];
-
-    } else if (inputArray.length === 2) {
-        name = inputArray[0];
-        branch = inputArray[1];
-
-    } else {
-        throw new Error("Invalid format. The format is like this: mhkarimi1383:goalert-notification");
-    }
-
-    return {
-        name,
-        branch
-    };
-}

package-lock.json

@@ -1,12 +1,12 @@
 {
     "name": "uptime-kuma",
-    "version": "2.0.0-beta.3",
+    "version": "2.0.0-beta.2",
     "lockfileVersion": 3,
     "requires": true,
     "packages": {
         "": {
             "name": "uptime-kuma",
-            "version": "2.0.0-beta.3",
+            "version": "2.0.0-beta.2",
             "license": "MIT",
             "dependencies": {
                 "@grpc/grpc-js": "~1.8.22",

package.json

@@ -57,7 +57,7 @@
         "release-nightly": "node ./extra/release/nightly.mjs",
         "git-remove-tag": "git tag -d",
         "build-dist-and-restart": "npm run build && npm run start-server-dev",
-        "start-pr-test": "node extra/checkout-pr.mjs && npm install && npm run dev",
+        "start-pr-test": "node extra/checkout-pr.js && npm install && npm run dev",
         "build-healthcheck-armv7": "cross-env GOOS=linux GOARCH=arm GOARM=7 go build -x -o ./extra/healthcheck-armv7 ./extra/healthcheck.go",
         "deploy-demo-server": "node extra/deploy-demo-server.js",
         "sort-contributors": "node extra/sort-contributors.js",

server/database.js

@@ -1,5 +1,4 @@
 const fs = require("fs");
-const fsAsync = fs.promises;
 const { R } = require("redbean-node");
 const { setSetting, setting } = require("./util-server");
 const { log, sleep } = require("../src/util");
@@ -708,12 +707,12 @@ class Database {
 
     /**
      * Get the size of the database (SQLite only)
-     * @returns {Promise<number>} Size of database
+     * @returns {number} Size of database
      */
-    static async getSize() {
+    static getSize() {
         if (Database.dbConfig.type === "sqlite") {
             log.debug("db", "Database.getSize()");
-            let stats = await fsAsync.stat(Database.sqlitePath);
+            let stats = fs.statSync(Database.sqlitePath);
             log.debug("db", stats);
             return stats.size;
         }

server/monitor-types/real-browser-monitor-type.js

@@ -2,13 +2,13 @@ const { MonitorType } = require("./monitor-type");
 const { chromium } = require("playwright-core");
 const { UP, log } = require("../../src/util");
 const { Settings } = require("../settings");
+const commandExistsSync = require("command-exists").sync;
 const childProcess = require("child_process");
 const path = require("path");
 const Database = require("../database");
 const jwt = require("jsonwebtoken");
 const config = require("../config");
 const { RemoteBrowser } = require("../remote-browser");
-const { commandExists } = require("../util-server");
 
 /**
  * Cached instance of a browser
@@ -122,7 +122,7 @@ async function prepareChromeExecutable(executablePath) {
             executablePath = "/usr/bin/chromium";
 
             // Install chromium in container via apt install
-            if (! await commandExists(executablePath)) {
+            if ( !commandExistsSync(executablePath)) {
                 await new Promise((resolve, reject) => {
                     log.info("Chromium", "Installing Chromium...");
                     let child = childProcess.exec("apt update && apt --yes --no-install-recommends install chromium fonts-indic fonts-noto fonts-noto-cjk");
@@ -146,7 +146,7 @@ async function prepareChromeExecutable(executablePath) {
             }
 
         } else {
-            executablePath = await findChrome(allowedList);
+            executablePath = findChrome(allowedList);
         }
     } else {
         // User specified a path
@@ -160,20 +160,20 @@ async function prepareChromeExecutable(executablePath) {
 
 /**
  * Find the chrome executable
- * @param {string[]} executables Executables to search through
- * @returns {Promise<string>} Executable
- * @throws {Error} Could not find executable
+ * @param {any[]} executables Executables to search through
+ * @returns {any} Executable
+ * @throws Could not find executable
  */
-async function findChrome(executables) {
+function findChrome(executables) {
     // Use the last working executable, so we don't have to search for it again
     if (lastAutoDetectChromeExecutable) {
-        if (await commandExists(lastAutoDetectChromeExecutable)) {
+        if (commandExistsSync(lastAutoDetectChromeExecutable)) {
             return lastAutoDetectChromeExecutable;
         }
     }
 
     for (let executable of executables) {
-        if (await commandExists(executable)) {
+        if (commandExistsSync(executable)) {
             lastAutoDetectChromeExecutable = executable;
             return executable;
         }

server/notification.js

@@ -77,7 +77,6 @@ const SendGrid = require("./notification-providers/send-grid");
 const YZJ = require("./notification-providers/yzj");
 const SMSPlanet = require("./notification-providers/sms-planet");
 const SpugPush = require("./notification-providers/spugpush");
-const { commandExists } = require("./util-server");
 
 class Notification {
 
@@ -260,10 +259,12 @@ class Notification {
 
     /**
      * Check if apprise exists
-     * @returns {Promise<boolean>} Does the command apprise exist?
+     * @returns {boolean} Does the command apprise exist?
      */
-    static async checkApprise() {
-        return await commandExists("apprise");
+    static checkApprise() {
+        let commandExistsSync = require("command-exists").sync;
+        let exists = commandExistsSync("apprise");
+        return exists;
     }
 
 }

server/server.js

@@ -1490,7 +1490,7 @@ let needSetup = false;
         socket.on("checkApprise", async (callback) => {
             try {
                 checkLogin(socket);
-                callback(await Notification.checkApprise());
+                callback(Notification.checkApprise());
             } catch (e) {
                 callback(false);
             }

server/socket-handlers/database-socket-handler.js

@@ -14,7 +14,7 @@ module.exports.databaseSocketHandler = (socket) => {
             checkLogin(socket);
             callback({
                 ok: true,
-                size: await Database.getSize(),
+                size: Database.getSize(),
             });
         } catch (error) {
             callback({

server/socket-handlers/general-socket-handler.js

@@ -4,7 +4,7 @@ const { sendInfo } = require("../client");
 const { checkLogin } = require("../util-server");
 const GameResolver = require("gamedig/lib/GameResolver");
 const { testChrome } = require("../monitor-types/real-browser-monitor-type");
-const fsAsync = require("fs").promises;
+const fs = require("fs");
 const path = require("path");
 
 let gameResolver = new GameResolver();
@@ -90,7 +90,7 @@ module.exports.generalSocketHandler = (socket, server) => {
         }
     });
 
-    socket.on("getPushExample", async (language, callback) => {
+    socket.on("getPushExample", (language, callback) => {
         try {
             checkLogin(socket);
             if (!/^[a-z-]+$/.test(language)) {
@@ -106,13 +106,13 @@ module.exports.generalSocketHandler = (socket, server) => {
 
         try {
             let dir = path.join("./extra/push-examples", language);
-            let files = await fsAsync.readdir(dir);
+            let files = fs.readdirSync(dir);
 
             for (let file of files) {
                 if (file.startsWith("index.")) {
                     callback({
                         ok: true,
-                        code: await fsAsync.readFile(path.join(dir, file), "utf8"),
+                        code: fs.readFileSync(path.join(dir, file), "utf8"),
                     });
                     return;
                 }

server/util-server.js

@@ -1096,19 +1096,3 @@ module.exports.axiosAbortSignal = (timeoutMs) => {
         }
     }
 };
-
-/**
- * By default, command-exists will throw a null error if the command does not exist, which is ugly. The function makes it better.
- * Read more: https://github.com/mathisonian/command-exists/issues/22
- * @param {string} command Command to check
- * @returns {Promise<boolean>} True if command exists, false otherwise
- */
-async function commandExists(command) {
-    try {
-        await require("command-exists")(command);
-        return true;
-    } catch (e) {
-        return false;
-    }
-}
-module.exports.commandExists = commandExists;

src/i18n.js

@@ -75,20 +75,11 @@ export function currentLocale() {
         if (locale in messages) {
             return locale;
         }
-        // If the locale is a 2-letter code, we can try to find a regional variant
-        // e.g. "fr" may not be in the messages, but "fr-FR" is
-        if (locale.length === 2) {
-            const regionalLocale = `${locale}-${locale.toUpperCase()}`;
-            if (regionalLocale in messages) {
-                return regionalLocale;
-            }
-        } else {
-            // Some locales are further specified such as "en-US".
-            // If we only have a generic locale for this, we can use it too
-            const genericLocale = locale.slice(0, 2);
-            if (genericLocale in messages) {
-                return genericLocale;
-            }
+        // some locales are further specified such as "en-US".
+        // If we only have a generic locale for this, we can use it too
+        const genericLocale = locale.split("-")[0];
+        if (genericLocale in messages) {
+            return genericLocale;
         }
     }
     return "en";

src/layouts/Layout.vue

@@ -16,7 +16,7 @@
                 <span class="fs-4 title">{{ $t("Uptime Kuma") }}</span>
             </router-link>
 
-            <a v-if="hasNewVersion" target="_blank" href="https://github.com/louislam/uptime-kuma/releases" class="btn btn-primary me-3">
+            <a v-if="hasNewVersion" target="_blank" href="https://github.com/louislam/uptime-kuma/releases" class="btn btn-info me-3">
                 <font-awesome-icon icon="arrow-alt-circle-up" /> {{ $t("New Update") }}
             </a>
 

src/pages/Details.vue

@@ -9,8 +9,7 @@
                     <div>{{ monitor.id }}</div>
                 </div>
             </h1>
-            <!-- eslint-disable-next-line vue/no-v-html-->
-            <p v-if="monitor.description" v-html="descriptionHTML"></p>
+            <p v-if="monitor.description">{{ monitor.description }}</p>
             <div class="d-flex">
                 <div class="tags">
                     <Tag v-for="tag in monitor.tags" :key="tag.id" :item="tag" :size="'sm'" />
@@ -286,8 +285,6 @@ import Tag from "../components/Tag.vue";
 import CertificateInfo from "../components/CertificateInfo.vue";
 import { getMonitorRelativeURL } from "../util.ts";
 import { URL } from "whatwg-url";
-import DOMPurify from "dompurify";
-import { marked } from "marked";
 import { getResBaseURL } from "../util-frontend";
 import { highlight, languages } from "prismjs/components/prism-core";
 import "prismjs/components/prism-clike";
@@ -402,14 +399,6 @@ export default {
 
         screenshotURL() {
             return getResBaseURL() + this.monitor.screenshot + "?time=" + this.cacheTime;
-        },
-
-        descriptionHTML() {
-            if (this.monitor.description != null) {
-                return DOMPurify.sanitize(marked(this.monitor.description));
-            } else {
-                return "";
-            }
         }
     },
 

src/pages/StatusPage.vue

@@ -157,12 +157,12 @@
             <!-- Admin functions -->
             <div v-if="hasToken" class="mb-4">
                 <div v-if="!enableEditMode">
-                    <button class="btn btn-primary me-2" data-testid="edit-button" @click="edit">
+                    <button class="btn btn-info me-2" data-testid="edit-button" @click="edit">
                         <font-awesome-icon icon="edit" />
                         {{ $t("Edit Status Page") }}
                     </button>
 
-                    <a href="/manage-status-page" class="btn btn-primary">
+                    <a href="/manage-status-page" class="btn btn-info">
                         <font-awesome-icon icon="tachometer-alt" />
                         {{ $t("Go to Dashboard") }}
                     </a>