@raketa-cloud/admin 0.0.93 → 0.0.95

package/jest.config.js

@@ -11,8 +11,8 @@ const config = {
   ],
   coverageThreshold: {
     global: {
-      branches: 71,
-      functions: 74,
+      branches: 72,
+      functions: 75,
       lines: 78,
       statements: 78,
     },

package/package.json

@@ -1,10 +1,10 @@
 {
   "name": "@raketa-cloud/admin",
-  "version": "0.0.93",
+  "version": "0.0.95",
   "main": "src/index.js",
   "scripts": {
     "dev": "npx tailwindcss -i ./src/styles/application.css -o ../../apps/admin/static/application.css --watch",
-    "test": "jest --coverage --coverageReporters=text --coverageReporters=\"text-summary\""
+    "test": "jest --coverage --coverageReporters=html --coverageReporters=\"text-summary\""
   },
   "devDependencies": {
     "@babel/core": "7.24.7",

package/src/handlers/RedirectsHandler.js

@@ -102,7 +102,7 @@ const destroy = async (ctx) => {
   const { errors } = await ctx.client.redirects.delete(id);
 
   if (errors) {
-    throw `Failed to delete redirect #${id}. WTF?`;
+    throw `Failed to delete redirect #${id}.`;
   }
 
   return {

package/src/handlers/TagsHandler.js

@@ -92,7 +92,7 @@ const destroy = async (ctx) => {
   const { errors } = await ctx.client.tags.delete(id, ctx.locale);
 
   if (errors) {
-    throw `Failed to delete tag #${id}. WTF?`;
+    throw `Failed to delete tag #${id}.`;
   }
 
   return {

package/src/handlers/categories/index.js

@@ -106,7 +106,7 @@ const destroy = async (ctx) => {
 
   return {
     headers: {
-      "HX-redirect": `/${ctx.locale}/${ctx.resourceConfig.categoryTypePlural}`,
+      "HX-Redirect": `/${ctx.locale}/${ctx.resourceConfig.categoryTypePlural}`,
     },
     json: {},
   };

package/src/helpers/HTMLBuilder.js

@@ -31,7 +31,7 @@ export default class HTMLBuilder {
     }, "");
 
     if (children || nonVoid)
-      return `<${tag} ${htmlAttributes}>${children}</${tag}>`;
+      return `<${tag} ${htmlAttributes}>${children ? children : ""}</${tag}>`;
 
     return `<${tag} ${htmlAttributes} />`;
   }

package/src/setupAuthentication.js

@@ -7,26 +7,28 @@ import AuthHelper from "./helpers/AuthHelper";
 import simpleCipher from "./simpleCipher";
 
 // Generate a random signing secret on every server restart
-const SIGNING_SECRET = randomBytes(64).toString("hex");
+// const SIGNING_SECRET = randomBytes(64).toString("hex");
 const DELIMITER = ",";
 
-function calculateHash(text) {
-  return createHmac("sha256", SIGNING_SECRET).update(text).digest("hex");
+function calculateHash(text, config) {
+  return createHmac("sha256", config.secrets.signing_secret)
+    .update(text)
+    .digest("hex");
 }
 
-function obfuscateToken(token) {
+function obfuscateToken(token, config) {
   const date = new Date();
   const timestamp = date.getTime() + 2 * 60000;
   const timestamped = [token, timestamp].join(DELIMITER);
-  const hash = calculateHash(timestamped);
+  const hash = calculateHash(timestamped, config);
   const signed = [hash, timestamped].join(DELIMITER);
 
-  return Buffer.from(simpleCipher.encrypt(signed)).toString("base64");
+  return Buffer.from(simpleCipher.encrypt(signed, config)).toString("base64");
 }
 
-function extractToken(text) {
+function extractToken(text, config) {
   const encrypted = Buffer.from(text, "base64").toString("ascii");
-  const signed = simpleCipher.decrypt(encrypted);
+  const signed = simpleCipher.decrypt(encrypted, config);
   const [hash, token, timestamp] = signed.split(DELIMITER);
   const tDate = new Date(Number.parseInt(timestamp, 10));
 
@@ -34,7 +36,7 @@ function extractToken(text) {
     return null;
   }
 
-  if (hash !== calculateHash([token, timestamp].join(DELIMITER))) {
+  if (hash !== calculateHash([token, timestamp].join(DELIMITER), config)) {
     return null;
   }
 
@@ -138,9 +140,7 @@ const setupAuthentication = (app, config) => {
       return c.redirect("/login");
     }
 
-    const ot = obfuscateToken(token);
-
-    console.log({ ot });
+    const ot = obfuscateToken(token, config);
 
     return c.redirect(
       [
@@ -148,7 +148,7 @@ const setupAuthentication = (app, config) => {
         "?redirect_to=",
         c.req.query().redirect_to,
         "&snapshot=",
-        obfuscateToken(token),
+        ot,
       ].join("")
     );
   });
@@ -156,7 +156,7 @@ const setupAuthentication = (app, config) => {
   app.post("/remote/verify", async (c) => {
     const { credentials } = await c.req.json();
 
-    const token = extractToken(credentials);
+    const token = extractToken(credentials, config);
 
     if (!token) {
       return unauthorized();

package/src/simpleCipher.js

@@ -1,14 +1,6 @@
 const ALPHABET =
   "ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz0123456789,.";
 
-function generateKey() {
-  const shuffled = ALPHABET.split("")
-    .sort(() => Math.random() - 0.5)
-    .join("");
-
-  return shuffled;
-}
-
 function generateEncryptionLookup(key) {
   const lookup = {};
 
@@ -29,22 +21,34 @@ function generateDecryptionLookup(key) {
   return lookup;
 }
 
+// function generateKey() {
+//   const shuffled = ALPHABET.split("")
+//     .sort(() => Math.random() - 0.5)
+//     .join("");
+
+//   return shuffled;
+// }
+
 // Scramble lookup maps on every server restart
-const CIPHER_KEY = generateKey();
-const ENCRYPT_LOOKUP = generateEncryptionLookup(CIPHER_KEY);
-const DECRYPT_LOOKUP = generateDecryptionLookup(CIPHER_KEY);
+// const CIPHER_KEY = generateKey();
+// const ENCRYPT_LOOKUP = generateEncryptionLookup(CIPHER_KEY);
+// const DECRYPT_LOOKUP = generateDecryptionLookup(CIPHER_KEY);
+
+function encrypt(plaintext, config) {
+  const ecryptLookup = generateEncryptionLookup(config.secrets.crypto_secret);
 
-function encrypt(plaintext) {
   return plaintext
     .split("")
-    .map((char) => ENCRYPT_LOOKUP[char] || char)
+    .map((char) => ecryptLookup[char] || char)
     .join("");
 }
 
-function decrypt(ciphertext) {
+function decrypt(ciphertext, config) {
+  const decryptLookup = generateDecryptionLookup(config.secrets.crypto_secret);
+
   return ciphertext
     .split("")
-    .map((char) => DECRYPT_LOOKUP[char] || char)
+    .map((char) => decryptLookup[char] || char)
     .join("");
 }
 

package/src/tests/Logger.test.js

@@ -0,0 +1,33 @@
+import Logger from "../Logger";
+
+describe("Logger", () => {
+  describe("error", () => {
+    test("Logs an error to the console", () => {
+      const logSpy = jest.spyOn(global.console, "error");
+
+      Logger.error({ foo: "BAR" });
+
+      expect(logSpy).toHaveBeenCalledWith("[ERROR]", "{ foo: 'BAR' }");
+    });
+  });
+
+  describe("info", () => {
+    test("Logs info to the console", () => {
+      const logSpy = jest.spyOn(global.console, "info");
+
+      Logger.info({ foo: "BAR" });
+
+      expect(logSpy).toHaveBeenCalledWith("[INFO]", "{ foo: 'BAR' }");
+    });
+  });
+
+  describe("debug", () => {
+    test("Logs debug information to the console", () => {
+      const logSpy = jest.spyOn(global.console, "debug");
+
+      Logger.debug({ foo: "BAR" });
+
+      expect(logSpy).toHaveBeenCalledWith("[DEBUG]", "{ foo: 'BAR' }");
+    });
+  });
+});

package/src/tests/handlers/RedirectsHandler.test/destroy.test.js

@@ -45,7 +45,7 @@ describe("Redirects", () => {
       }));
 
     await expect(() => RedirectsHandler.destroy(ctx)).rejects.toMatch(
-      "Failed to delete redirect #1. WTF?"
+      "Failed to delete redirect #1."
     );
 
     expect(mockDelete).toHaveBeenCalledWith(1);

package/src/tests/handlers/TagsHandler.test/destroy.test.js

@@ -43,7 +43,7 @@ describe("Tags Handler", () => {
       }));
 
     expect(() => TagsHandler.destroy(ctx)).rejects.toMatch(
-      "Failed to delete tag #1. WTF?"
+      "Failed to delete tag #1."
     );
 
     expect(mockedDelete).toHaveBeenCalledWith("1", "en");

package/src/tests/handlers/categories/destroy.test.js

@@ -1,34 +1,38 @@
-import RedirectsHandler from "../../../handlers/RedirectsHandler";
-import Redirects from "../../../cloud/Redirects";
+import CategoriesHandler from "../../../handlers/categories";
+import Categories from "../../../cloud/Categories";
 
 import Factory from "../../support/factory";
 
 function buildCtx() {
   return {
     client: {
-      redirects: new Redirects(() => ({}), "token"),
+      categories: new Categories(() => ({}), "token"),
     },
     params: { id: 1 },
     locale: "en",
+    resourceConfig: {
+      categoryType: "cat",
+      categoryTypePlural: "cats",
+    },
   };
 }
 
-describe("Redirects", () => {
+describe("Categories", () => {
   test("destroy successful", async () => {
     const redirect = Factory.build("redirect");
     const ctx = buildCtx();
 
     const mockDelete = jest
-      .spyOn(ctx.client.redirects, "delete")
+      .spyOn(ctx.client.categories, "delete")
       .mockImplementation(() => ({ redirect: redirect, errors: null }));
 
-    const result = await RedirectsHandler.destroy(ctx);
+    const result = await CategoriesHandler.destroy(ctx);
 
-    expect(mockDelete).toHaveBeenCalledWith(1);
+    expect(mockDelete).toHaveBeenCalledWith(1, "en");
 
     expect(result).toEqual({
       headers: {
-        "HX-Redirect": "/en/redirects",
+        "HX-Redirect": "/en/cats",
       },
       json: {},
     });
@@ -38,16 +42,16 @@ describe("Redirects", () => {
     const ctx = buildCtx();
 
     const mockDelete = jest
-      .spyOn(ctx.client.redirects, "delete")
+      .spyOn(ctx.client.categories, "delete")
       .mockImplementation(() => ({
         redirect: null,
         errors: [{ key: "generalError", message: "Not found" }],
       }));
 
-    await expect(() => RedirectsHandler.destroy(ctx)).rejects.toMatch(
-      "Failed to delete redirect #1. WTF?"
+    await expect(() => CategoriesHandler.destroy(ctx)).rejects.toEqual(
+      "Failed to delete category #1."
     );
 
-    expect(mockDelete).toHaveBeenCalledWith(1);
+    expect(mockDelete).toHaveBeenCalledWith(1, "en");
   });
 });

package/src/tests/helpers/HTMLBuilder.test.js

@@ -0,0 +1,44 @@
+import HTMLBuilder from "../../helpers/HTMLBuilder";
+
+describe("HTMLBuilder", () => {
+  function normalizeHTML(html) {
+    return html.replace(/>\s+</g, "><").replace(/\s+/g, " ").trim();
+  }
+
+  describe("tag", () => {
+    test("renders a self closing tag", () => {
+      const builder = new HTMLBuilder();
+
+      const result = builder.tag("img", null, { src: "/some-image.jpg" });
+
+      expect(normalizeHTML(result)).toEqual(
+        normalizeHTML('<img src="/some-image.jpg" />')
+      );
+    });
+
+    test("renders a label tag", () => {
+      const builder = new HTMLBuilder();
+
+      const result = builder.tag("label", "Heelo", { for: "some-input-id" });
+
+      expect(normalizeHTML(result)).toEqual(
+        normalizeHTML('<label for="some-input-id">Heelo</label>')
+      );
+    });
+
+    test("renders a textarea tag", () => {
+      const builder = new HTMLBuilder();
+
+      const result = builder.tag(
+        "textarea",
+        undefined,
+        { id: "some-input-id", value: "Text?" },
+        { nonVoid: true }
+      );
+
+      expect(normalizeHTML(result)).toEqual(
+        normalizeHTML('<textarea id="some-input-id" value="Text?"></textarea>')
+      );
+    });
+  });
+});