@flowselections/layout-maker 1.54.1 → 1.55.1

package/dist-lib/lib/printnodeWebhook.js

@@ -16,108 +16,134 @@ function gelijk(a, b) {
         verschil |= a.charCodeAt(i) ^ b.charCodeAt(i);
     return verschil === 0;
 }
-/**
- * PrintNode documenteert alleen dat het secret "in each request" wordt
- * meegestuurd, niet in welk veld. Daarom: is het veld nog niet bekend, dan
- * zoeken we het ENE kopveld waarvan de waarde (in constante tijd) gelijk is
- * aan het secret en leggen alleen die naam vast. Daarna telt alleen dat veld.
- */
-export function controleerSecret(request, secret, veld) {
-    if (veld) {
-        const w = request.headers.get(veld);
-        return { ok: w != null && gelijk(w.trim(), secret), veld };
-    }
-    const treffers = [];
-    request.headers.forEach((waarde, naam) => {
-        if (gelijk(waarde.trim(), secret))
-            treffers.push(naam.toLowerCase());
-    });
-    return treffers.length === 1 ? { ok: true, veld: treffers[0] } : { ok: false, veld: null };
+/** De kop waarin PrintNode het geheim meestuurt (hulptekst webhookpagina). */
+export const SECRET_KOP = "x-printnode-webhook-secret";
+/** Controleert alleen de kop X-PrintNode-Webhook-Secret, in constante tijd. */
+export function controleerSecret(request, secret, _veld) {
+    const w = request.headers.get(SECRET_KOP);
+    return { ok: w != null && gelijk(w.trim(), secret), veld: SECRET_KOP };
 }
 /** @deprecated gebruik controleerSecret */
 export function secretKlopt(request, secret) {
-    return controleerSecret(request, secret, null).ok;
+    return controleerSecret(request, secret).ok;
+}
+const wacht = (ms) => new Promise((r) => setTimeout(r, ms));
+/** Zoekt de opdracht op PrintNode-id; kort opnieuw als het id nog niet is opgeslagen. */
+async function zoekJob(db, externId, pogingen = 3) {
+    for (let i = 0; i < pogingen; i++) {
+        const { data } = await db.from("print_jobs").select("id").eq("extern_job_id", externId).maybeSingle();
+        const id = data?.id;
+        if (id)
+            return id;
+        if (i < pogingen - 1)
+            await wacht(1500);
+    }
+    return null;
+}
+async function noteer(db, regel, bewaren) {
+    if (!bewaren)
+        return;
+    try {
+        await db.from("printnode_webhook_meldingen").insert(regel);
+        const { data } = await db.from("printnode_webhook_meldingen").select("id").order("id", { ascending: false }).range(19, 19);
+        const grens = data?.[0]?.id;
+        if (grens)
+            await db.from("printnode_webhook_meldingen").delete().lt("id", grens);
+    }
+    catch { /* vastleggen mag verwerking niet blokkeren */ }
 }
 /**
  * Afhandeling van een webhookaanroep. Het geheim komt uit de eigen
  * koppelingstabel (servicerol). `secretVoorTest` alleen voor tests.
+ * Elk geldig (geheim klopt) verzoek krijgt 2xx + OK-kop, ook bij onbekende
+ * opdrachten of een fout in één melding.
  */
-export async function behandelPrintNodeWebhook(request, db, secretVoorTest) {
+export async function behandelPrintNodeWebhook(request, db, secretVoorTest, opties = {}) {
     let secret = secretVoorTest;
-    let veld = null;
     const uitTabel = secretVoorTest === undefined;
     if (uitTabel) {
         try {
-            const { data } = await db.from("printnode_webhook_koppeling").select("geheim, secret_veld, status").eq("id", 1).maybeSingle();
+            const { data } = await db.from("printnode_webhook_koppeling").select("geheim, status").eq("id", 1).maybeSingle();
             const k = data;
-            if (k?.status === "aan" && k.geheim) {
+            if (k?.status === "aan" && k.geheim)
                 secret = k.geheim;
-                veld = k.secret_veld ?? null;
-            }
         }
         catch { /* fail closed */ }
     }
-    // Fail closed: zonder koppeling wordt niets verwerkt.
     if (!secret) {
         return Response.json({ ok: false, fout: "Live printstatus is niet gekoppeld." }, { status: 503 });
     }
-    const c = controleerSecret(request, secret, veld);
-    if (!c.ok)
+    if (!controleerSecret(request, secret).ok) {
+        await noteer(db, { http_status: 401, antwoord: { fout: "Ongeldig secret" } }, uitTabel);
         return Response.json({ ok: false, fout: "Ongeldig secret" }, { status: 401 });
+    }
     if (uitTabel) {
         try {
             await db.from("printnode_webhook_koppeling").update({
-                laatst_ontvangen_op: new Date().toISOString(),
-                ...(veld ? {} : { secret_veld: c.veld }),
+                laatst_ontvangen_op: new Date().toISOString(), secret_veld: SECRET_KOP,
             }).eq("id", 1);
         }
-        catch { /* status bijhouden mag verwerking niet blokkeren */ }
+        catch { /* niet blokkeren */ }
     }
-    let body;
+    let berichten = [];
+    let jsonFout = false;
     try {
-        body = await request.json();
+        const body = await request.json();
+        berichten = Array.isArray(body) ? body : [];
     }
     catch {
-        return Response.json({ ok: false, fout: "Ongeldige JSON" }, { status: 400 });
+        jsonFout = true;
     }
-    const berichten = Array.isArray(body) ? body : [];
-    const { pasPrintNodeStatusToe } = await import("./printQueue.server.js");
+    const soorten = new Set();
+    const ids = [];
+    const nietGematcht = [];
+    const fouten = [];
     let jobs = 0;
     let computerBericht = false;
+    const pogingen = opties.wachtMs === 0 ? 1 : 3;
+    const { pasPrintNodeStatusToe } = await import("./printQueue.server.js");
     for (const b of berichten.slice(0, 500)) {
-        const type = String(b?.type ?? "").toLowerCase();
-        if (type === "print job state") {
-            const d = b.data ?? {};
-            const externId = d["printJobId"];
-            const state = String(d["state"] ?? "").toLowerCase();
-            if ((typeof externId !== "number" && typeof externId !== "string") || !state)
-                continue;
-            const { data: job } = await db
-                .from("print_jobs")
-                .select("id")
-                .eq("extern_job_id", String(externId))
-                .maybeSingle();
-            const id = job?.id;
-            if (!id)
-                continue;
-            await pasPrintNodeStatusToe(db, id, state, String(d["message"] ?? ""));
-            jobs++;
+        try {
+            const type = String(b?.type ?? "").toLowerCase();
+            if (type)
+                soorten.add(type);
+            if (type === "print job state") {
+                const d = b.data ?? {};
+                const externId = d["printJobId"];
+                const state = String(d["state"] ?? "").toLowerCase();
+                if ((typeof externId !== "number" && typeof externId !== "string") || !state)
+                    continue;
+                ids.push(`${externId}:${state}`);
+                const id = await zoekJob(db, String(externId), pogingen);
+                if (!id) {
+                    nietGematcht.push(String(externId));
+                    continue;
+                }
+                await pasPrintNodeStatusToe(db, id, state, String(d["message"] ?? ""));
+                jobs++;
+            }
+            else if (type === "computer state") {
+                computerBericht = true;
+            }
         }
-        else if (type === "computer state") {
-            computerBericht = true;
+        catch (e) {
+            fouten.push(e instanceof Error ? e.message.slice(0, 200) : "onbekend");
         }
     }
-    // Een computerbericht bevat geen computer-id (alleen hostname). Om op id te
-    // matchen halen we de actuele lijst op en slaan die op per computer-id.
     if (computerBericht) {
         try {
             await ververs(db);
         }
         catch (e) {
-            console.warn("[printnode-webhook] computerstatus verversen mislukt:", e);
+            fouten.push(`computerstatus: ${e instanceof Error ? e.message.slice(0, 150) : "onbekend"}`);
         }
     }
-    return Response.json({ ok: true, jobs, computers: computerBericht }, { headers: OK_KOP });
+    const antwoord = { ok: true, jobs, computers: computerBericht, onbekend: nietGematcht.length, ...(jsonFout ? { waarschuwing: "Ongeldige JSON" } : {}), ...(fouten.length ? { fouten } : {}) };
+    await noteer(db, {
+        http_status: 200, soorten: [...soorten], printnode_ids: ids.slice(0, 50),
+        gematcht: jobs, niet_gematcht: nietGematcht.slice(0, 50), antwoord,
+    }, uitTabel);
+    return Response.json(antwoord, { headers: OK_KOP });
 }
 /** Haalt alle computers op bij PrintNode en slaat hun status op per id. */
 export async function ververs(db) {

package/dist-lib/lib/printnodeWebhookBeheer.server.d.ts

@@ -19,6 +19,8 @@ export interface Koppeling {
     pn_mislukt?: number | null;
     pn_geslaagd_vorig?: number | null;
     pn_opgehaald_op?: string | null;
+    pn_ontvangen?: number | null;
+    pn_weggegooid?: number | null;
 }
 export declare function leesKoppeling(db: Db): Promise<Koppeling | null>;
 /** Het Shell-adres zoals de Logistiek Engine het bewaart en controleert. */
@@ -45,10 +47,12 @@ export type ZelftestUitkomst = "ja" | "nee" | "onbekend";
  * JSON (401/503) = route bestaat. Alleen een echte 404 = niet bereikbaar.
  */
 export declare function beoordeelZelftest(status: number, contentType: string | null): ZelftestUitkomst;
-/** Leest geslaagde/mislukte aantallen uit `counts`; zonder herkenbaar veld: null. */
+/** Leest counts uit GET /webhooks (successfulRequests, failedRequests, receivedEvents, droppedEvents). */
 export declare function leesCounts(counts: unknown): {
     geslaagd: number | null;
     mislukt: number | null;
+    ontvangen: number | null;
+    weggegooid: number | null;
 };
 /** Zelftest naar het eigen adres + tellingen uit GET /webhooks (op webhookId). */
 export declare function controleerBereikbaarheid(db: Db): Promise<void>;
@@ -62,6 +66,8 @@ export declare function veiligeStatus(k: Koppeling | null): {
     pnUrl: string | null;
     pnGeslaagd: number | null;
     pnMislukt: number | null;
+    pnOntvangen: number | null;
+    pnWeggegooid: number | null;
     pnOpgehaaldOp: string | null;
     url: string | null;
     fout: string | null;

package/dist-lib/lib/printnodeWebhookBeheer.server.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"printnodeWebhookBeheer.server.d.ts","sourceRoot":"","sources":["../../src/lib/printnodeWebhookBeheer.server.ts"],"names":[],"mappings":"AAOA,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,uBAAuB,CAAC;AAG5D,KAAK,EAAE,GAAG,cAAc,CAAC,GAAG,EAAE,GAAG,EAAE,GAAG,CAAC,CAAC;AAQxC,MAAM,WAAW,SAAS;IACxB,MAAM,EAAE,MAAM,GAAG,IAAI,CAAC;IACtB,WAAW,EAAE,MAAM,GAAG,IAAI,CAAC;IAC3B,UAAU,EAAE,MAAM,GAAG,IAAI,CAAC;IAC1B,GAAG,EAAE,MAAM,GAAG,IAAI,CAAC;IACnB,MAAM,EAAE,MAAM,CAAC;IACf,IAAI,EAAE,MAAM,GAAG,IAAI,CAAC;IACpB,OAAO,EAAE,MAAM,GAAG,IAAI,CAAC;IACvB,YAAY,EAAE,MAAM,GAAG,IAAI,CAAC;IAC5B,iBAAiB,EAAE,MAAM,GAAG,IAAI,CAAC;IACjC,mBAAmB,EAAE,MAAM,GAAG,IAAI,CAAC;IACnC,UAAU,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;IAC3B,WAAW,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;IAC5B,aAAa,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;IAC9B,MAAM,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;IACvB,WAAW,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;IAC5B,UAAU,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;IAC3B,iBAAiB,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;IAClC,eAAe,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;CACjC;AAID,wBAAsB,aAAa,CAAC,EAAE,EAAE,EAAE,GAAG,OAAO,CAAC,SAAS,GAAG,IAAI,CAAC,CAGrE;AAMD,4EAA4E;AAC5E,wBAAsB,cAAc,CAAC,EAAE,EAAE,EAAE,GAAG,OAAO,CAAC,MAAM,GAAG,IAAI,CAAC,CAInE;AAQD,MAAM,WAAW,cAAc;IAAG,EAAE,EAAE,OAAO,CAAC;IAAC,KAAK,CAAC,EAAE,MAAM,CAAC;IAAC,GAAG,CAAC,EAAE,MAAM,CAAC;IAAC,UAAU,CAAC,EAAE,OAAO,CAAA;CAAE;AAEnG,0DAA0D;AAC1D,wBAAsB,aAAa,CAAC,EAAE,EAAE,EAAE,EAAE,MAAM,GAAE;IAAE,SAAS,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;IAAC,OAAO,CAAC,EAAE,MAAM,GAAG,IAAI,CAAA;CAAO,GAAG,OAAO,CAAC,cAAc,CAAC,CA6CxI;AAED;;;GAGG;AACH,wBAAsB,wBAAwB,CAAC,EAAE,EAAE,EAAE,GAAG,OAAO,CAAC,IAAI,CAAC,CAsBpE;AAKD,MAAM,MAAM,gBAAgB,GAAG,IAAI,GAAG,KAAK,GAAG,UAAU,CAAC;AAEzD;;;GAGG;AACH,wBAAgB,iBAAiB,CAAC,MAAM,EAAE,MAAM,EAAE,WAAW,EAAE,MAAM,GAAG,IAAI,GAAG,gBAAgB,CAK9F;AAED,qFAAqF;AACrF,wBAAgB,UAAU,CAAC,MAAM,EAAE,OAAO,GAAG;IAAE,QAAQ,EAAE,MAAM,GAAG,IAAI,CAAC;IAAC,OAAO,EAAE,MAAM,GAAG,IAAI,CAAA;CAAE,CAU/F;AAED,kFAAkF;AAClF,wBAAsB,wBAAwB,CAAC,EAAE,EAAE,EAAE,GAAG,OAAO,CAAC,IAAI,CAAC,CAyCpE;AAED,0DAA0D;AAC1D,wBAAgB,cAAc,CAAC,CAAC,EAAE,SAAS,GAAG,IAAI,GAAG,KAAK,GAAG,KAAK,GAAG,UAAU,CAM9E;AAED,wDAAwD;AACxD,wBAAgB,aAAa,CAAC,CAAC,EAAE,SAAS,GAAG,IAAI;;;;;;;;;;;;;;EAiBhD"}
+{"version":3,"file":"printnodeWebhookBeheer.server.d.ts","sourceRoot":"","sources":["../../src/lib/printnodeWebhookBeheer.server.ts"],"names":[],"mappings":"AAOA,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,uBAAuB,CAAC;AAG5D,KAAK,EAAE,GAAG,cAAc,CAAC,GAAG,EAAE,GAAG,EAAE,GAAG,CAAC,CAAC;AAQxC,MAAM,WAAW,SAAS;IACxB,MAAM,EAAE,MAAM,GAAG,IAAI,CAAC;IACtB,WAAW,EAAE,MAAM,GAAG,IAAI,CAAC;IAC3B,UAAU,EAAE,MAAM,GAAG,IAAI,CAAC;IAC1B,GAAG,EAAE,MAAM,GAAG,IAAI,CAAC;IACnB,MAAM,EAAE,MAAM,CAAC;IACf,IAAI,EAAE,MAAM,GAAG,IAAI,CAAC;IACpB,OAAO,EAAE,MAAM,GAAG,IAAI,CAAC;IACvB,YAAY,EAAE,MAAM,GAAG,IAAI,CAAC;IAC5B,iBAAiB,EAAE,MAAM,GAAG,IAAI,CAAC;IACjC,mBAAmB,EAAE,MAAM,GAAG,IAAI,CAAC;IACnC,UAAU,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;IAC3B,WAAW,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;IAC5B,aAAa,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;IAC9B,MAAM,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;IACvB,WAAW,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;IAC5B,UAAU,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;IAC3B,iBAAiB,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;IAClC,eAAe,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;IAChC,YAAY,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;IAC7B,aAAa,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;CAC/B;AAID,wBAAsB,aAAa,CAAC,EAAE,EAAE,EAAE,GAAG,OAAO,CAAC,SAAS,GAAG,IAAI,CAAC,CAGrE;AAMD,4EAA4E;AAC5E,wBAAsB,cAAc,CAAC,EAAE,EAAE,EAAE,GAAG,OAAO,CAAC,MAAM,GAAG,IAAI,CAAC,CAInE;AAQD,MAAM,WAAW,cAAc;IAAG,EAAE,EAAE,OAAO,CAAC;IAAC,KAAK,CAAC,EAAE,MAAM,CAAC;IAAC,GAAG,CAAC,EAAE,MAAM,CAAC;IAAC,UAAU,CAAC,EAAE,OAAO,CAAA;CAAE;AAEnG,0DAA0D;AAC1D,wBAAsB,aAAa,CAAC,EAAE,EAAE,EAAE,EAAE,MAAM,GAAE;IAAE,SAAS,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;IAAC,OAAO,CAAC,EAAE,MAAM,GAAG,IAAI,CAAA;CAAO,GAAG,OAAO,CAAC,cAAc,CAAC,CA6CxI;AAED;;;GAGG;AACH,wBAAsB,wBAAwB,CAAC,EAAE,EAAE,EAAE,GAAG,OAAO,CAAC,IAAI,CAAC,CAsBpE;AAKD,MAAM,MAAM,gBAAgB,GAAG,IAAI,GAAG,KAAK,GAAG,UAAU,CAAC;AAEzD;;;GAGG;AACH,wBAAgB,iBAAiB,CAAC,MAAM,EAAE,MAAM,EAAE,WAAW,EAAE,MAAM,GAAG,IAAI,GAAG,gBAAgB,CAK9F;AAED,0GAA0G;AAC1G,wBAAgB,UAAU,CAAC,MAAM,EAAE,OAAO,GAAG;IAAE,QAAQ,EAAE,MAAM,GAAG,IAAI,CAAC;IAAC,OAAO,EAAE,MAAM,GAAG,IAAI,CAAC;IAAC,SAAS,EAAE,MAAM,GAAG,IAAI,CAAC;IAAC,UAAU,EAAE,MAAM,GAAG,IAAI,CAAA;CAAE,CASpJ;AAED,kFAAkF;AAClF,wBAAsB,wBAAwB,CAAC,EAAE,EAAE,EAAE,GAAG,OAAO,CAAC,IAAI,CAAC,CA2CpE;AAED,0DAA0D;AAC1D,wBAAgB,cAAc,CAAC,CAAC,EAAE,SAAS,GAAG,IAAI,GAAG,KAAK,GAAG,KAAK,GAAG,UAAU,CAM9E;AAED,wDAAwD;AACxD,wBAAgB,aAAa,CAAC,CAAC,EAAE,SAAS,GAAG,IAAI;;;;;;;;;;;;;;;;EAmBhD"}

package/dist-lib/lib/printnodeWebhookBeheer.server.js

@@ -121,18 +121,15 @@ export function beoordeelZelftest(status, contentType) {
         return "nee";
     return "onbekend";
 }
-/** Leest geslaagde/mislukte aantallen uit `counts`; zonder herkenbaar veld: null. */
+/** Leest counts uit GET /webhooks (successfulRequests, failedRequests, receivedEvents, droppedEvents). */
 export function leesCounts(counts) {
     const c = (counts && typeof counts === "object" ? counts : {});
-    const getal = (...k) => {
-        for (const n of k)
-            if (typeof c[n] === "number")
-                return c[n];
-        return null;
-    };
+    const n = (k) => (typeof c[k] === "number" ? c[k] : null);
     return {
-        geslaagd: getal("success", "successes", "succeeded", "ok", "delivered"),
-        mislukt: getal("failure", "failures", "failed", "error", "errors"),
+        geslaagd: n("successfulRequests"),
+        mislukt: n("failedRequests"),
+        ontvangen: n("receivedEvents"),
+        weggegooid: n("droppedEvents"),
     };
 }
 /** Zelftest naar het eigen adres + tellingen uit GET /webhooks (op webhookId). */
@@ -171,6 +168,8 @@ export async function controleerBereikbaarheid(db) {
                 velden.pn_geslaagd_vorig = k.pn_geslaagd ?? null;
                 velden.pn_geslaagd = c.geslaagd;
                 velden.pn_mislukt = c.mislukt;
+                velden.pn_ontvangen = c.ontvangen;
+                velden.pn_weggegooid = c.weggegooid;
                 velden.pn_opgehaald_op = nu;
                 if (c.geslaagd != null && k.pn_geslaagd != null && c.geslaagd > k.pn_geslaagd)
                     bereikbaar = "ja";
@@ -205,6 +204,8 @@ export function veiligeStatus(k) {
         pnUrl: k?.pn_url ?? null,
         pnGeslaagd: k?.pn_geslaagd ?? null,
         pnMislukt: k?.pn_mislukt ?? null,
+        pnOntvangen: k?.pn_ontvangen ?? null,
+        pnWeggegooid: k?.pn_weggegooid ?? null,
         pnOpgehaaldOp: k?.pn_opgehaald_op ?? null,
         url: k?.url ?? null,
         fout: k?.fout ?? null,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@flowselections/layout-maker",
-  "version": "1.54.1",
+  "version": "1.55.1",
   "flowselections": {
     "moduleId": "layout-maker",
     "pages": [