@camstack/addon-provider-reolink 1.2.161 → 1.2.162

package/dist/addon.js

@@ -25360,6 +25360,39 @@ _enum([
 _enum(["deferred", "final"]);
 /** The camera's / provider's own words. Kept verbatim, never parsed. */
 var CLIP_STILL_REASON_HEADER = "x-camstack-reason";
+/**
+* Make a refusal sentence safe to put in an HTTP header.
+*
+* **An HTTP header value is Latin-1.** Node throws `Invalid character in
+* header content` on anything above U+00FF, the handler dies mid-response,
+* and the client gets a socket hang up — no status, no reason, nothing. The
+* mechanism that exists to EXPLAIN a refusal is then the thing that silences
+* it, and the surface shows "nothing happened".
+*
+* That is not hypothetical: measured 2026-09-23 on 1436, a Hikvision replay
+* refusal read `SETUP of the video track answered 500 — the camera refused
+* this replay`, and the EM DASH in it threw on every clip the operator
+* opened. This repository writes its refusals in prose, and its prose uses
+* `—`, `’` and `…` everywhere, so every reason string is a candidate.
+*
+* Substitutes rather than strips: an operator reading a log or a response
+* should get a readable sentence, not one with holes in it. The typographic
+* characters this repo actually writes have exact ASCII equivalents; anything
+* else non-Latin-1 becomes `?`, which is visible rather than silent. Control
+* characters go too — a newline in a header value is a response-splitting
+* bug, not a formatting one.
+*/
+function clipReasonHeaderValue(reason) {
+	const folded = reason.replace(/[\u2013\u2014\u2212]/g, "-").replace(/[\u2018\u2019\u201b]/g, "'").replace(/[\u201c\u201d]/g, "\"").replace(/\u2026/g, "...").replace(/[\u00a0\u2007\u202f]/g, " ");
+	let out = "";
+	for (const ch of folded) {
+		const code = ch.codePointAt(0) ?? 0;
+		if (code < 32 || code === 127) out += " ";
+		else if (code > 255) out += "?";
+		else out += ch;
+	}
+	return out.slice(0, 500).trim();
+}
 /** One {@link ClipStillRefusalCodeSchema} token, alone. */
 var CLIP_STILL_REASON_CODE_HEADER = "x-camstack-reason-code";
 /** One {@link ClipStillDispositionSchema} token, alone. */
@@ -151601,7 +151634,7 @@ function writeRefusal(res, refusal, deviceId, logger, clipKey) {
 	else logger.debug(line, extras);
 	const retryAfterMs = refusal.disposition === "deferred" ? refusal.retryAfterMs : void 0;
 	res.writeHead(204, {
-		[CLIP_STILL_REASON_HEADER]: refusal.reason.slice(0, 500),
+		[CLIP_STILL_REASON_HEADER]: clipReasonHeaderValue(refusal.reason),
 		[CLIP_STILL_REASON_CODE_HEADER]: refusal.code,
 		[CLIP_STILL_DISPOSITION_HEADER]: refusal.disposition,
 		...retryAfterMs === void 0 ? {} : { "retry-after": String(Math.max(1, Math.ceil(retryAfterMs / 1e3))) }
@@ -151798,7 +151831,7 @@ function createClipStreamHandler(deps) {
 				status: resolved.status
 			} });
 			res.writeHead(resolved.status, {
-				[CLIP_STILL_REASON_HEADER]: resolved.detail.slice(0, 500),
+				[CLIP_STILL_REASON_HEADER]: clipReasonHeaderValue(resolved.detail),
 				[CLIP_STILL_REASON_CODE_HEADER]: resolved.code
 			});
 			res.end();
@@ -151871,7 +151904,7 @@ function createClipStreamHandler(deps) {
 			}
 		});
 		res.writeHead(streamRefusalStatus(outcome.code), {
-			[CLIP_STILL_REASON_HEADER]: outcome.detail.slice(0, 500),
+			[CLIP_STILL_REASON_HEADER]: clipReasonHeaderValue(outcome.detail),
 			[CLIP_STILL_REASON_CODE_HEADER]: outcome.code
 		});
 		res.end();
@@ -152313,7 +152346,33 @@ function createClipStreamProducer(deps) {
 				}
 			});
 			activeByHost.set(hostKey, ahead + 1);
-			const run = (lanes.get(hostKey) ?? Promise.resolve()).then(() => runAttempts(input)).finally(() => {
+			let abandonedAt = null;
+			input.sink.onceClose(() => {
+				abandonedAt ??= Date.now();
+			});
+			const queuedAt = Date.now();
+			const run = (lanes.get(hostKey) ?? Promise.resolve()).then(async () => {
+				if (abandonedAt !== null) {
+					const waitedMs = abandonedAt - queuedAt;
+					deps.logger.info("videoclips: a queued clip stream was dropped, its peer had gone", {
+						tags: { deviceId },
+						meta: {
+							clipKey: input.row.clipKey,
+							twin: input.twin,
+							branch: "queued-peer-gone",
+							waitedMs,
+							aheadWhenQueued: ahead
+						}
+					});
+					return {
+						kind: "refused",
+						code: "fetch-failed",
+						detail: `the peer left after ${String(waitedMs)} ms in the queue, so this clip was never fetched`,
+						attempts: 0
+					};
+				}
+				return await runAttempts(input);
+			}).finally(() => {
 				const left = (activeByHost.get(hostKey) ?? 1) - 1;
 				if (left <= 0) activeByHost.delete(hostKey);
 				else activeByHost.set(hostKey, left);

package/dist/addon.mjs

@@ -25355,6 +25355,39 @@ _enum([
 _enum(["deferred", "final"]);
 /** The camera's / provider's own words. Kept verbatim, never parsed. */
 var CLIP_STILL_REASON_HEADER = "x-camstack-reason";
+/**
+* Make a refusal sentence safe to put in an HTTP header.
+*
+* **An HTTP header value is Latin-1.** Node throws `Invalid character in
+* header content` on anything above U+00FF, the handler dies mid-response,
+* and the client gets a socket hang up — no status, no reason, nothing. The
+* mechanism that exists to EXPLAIN a refusal is then the thing that silences
+* it, and the surface shows "nothing happened".
+*
+* That is not hypothetical: measured 2026-09-23 on 1436, a Hikvision replay
+* refusal read `SETUP of the video track answered 500 — the camera refused
+* this replay`, and the EM DASH in it threw on every clip the operator
+* opened. This repository writes its refusals in prose, and its prose uses
+* `—`, `’` and `…` everywhere, so every reason string is a candidate.
+*
+* Substitutes rather than strips: an operator reading a log or a response
+* should get a readable sentence, not one with holes in it. The typographic
+* characters this repo actually writes have exact ASCII equivalents; anything
+* else non-Latin-1 becomes `?`, which is visible rather than silent. Control
+* characters go too — a newline in a header value is a response-splitting
+* bug, not a formatting one.
+*/
+function clipReasonHeaderValue(reason) {
+	const folded = reason.replace(/[\u2013\u2014\u2212]/g, "-").replace(/[\u2018\u2019\u201b]/g, "'").replace(/[\u201c\u201d]/g, "\"").replace(/\u2026/g, "...").replace(/[\u00a0\u2007\u202f]/g, " ");
+	let out = "";
+	for (const ch of folded) {
+		const code = ch.codePointAt(0) ?? 0;
+		if (code < 32 || code === 127) out += " ";
+		else if (code > 255) out += "?";
+		else out += ch;
+	}
+	return out.slice(0, 500).trim();
+}
 /** One {@link ClipStillRefusalCodeSchema} token, alone. */
 var CLIP_STILL_REASON_CODE_HEADER = "x-camstack-reason-code";
 /** One {@link ClipStillDispositionSchema} token, alone. */
@@ -151596,7 +151629,7 @@ function writeRefusal(res, refusal, deviceId, logger, clipKey) {
 	else logger.debug(line, extras);
 	const retryAfterMs = refusal.disposition === "deferred" ? refusal.retryAfterMs : void 0;
 	res.writeHead(204, {
-		[CLIP_STILL_REASON_HEADER]: refusal.reason.slice(0, 500),
+		[CLIP_STILL_REASON_HEADER]: clipReasonHeaderValue(refusal.reason),
 		[CLIP_STILL_REASON_CODE_HEADER]: refusal.code,
 		[CLIP_STILL_DISPOSITION_HEADER]: refusal.disposition,
 		...retryAfterMs === void 0 ? {} : { "retry-after": String(Math.max(1, Math.ceil(retryAfterMs / 1e3))) }
@@ -151793,7 +151826,7 @@ function createClipStreamHandler(deps) {
 				status: resolved.status
 			} });
 			res.writeHead(resolved.status, {
-				[CLIP_STILL_REASON_HEADER]: resolved.detail.slice(0, 500),
+				[CLIP_STILL_REASON_HEADER]: clipReasonHeaderValue(resolved.detail),
 				[CLIP_STILL_REASON_CODE_HEADER]: resolved.code
 			});
 			res.end();
@@ -151866,7 +151899,7 @@ function createClipStreamHandler(deps) {
 			}
 		});
 		res.writeHead(streamRefusalStatus(outcome.code), {
-			[CLIP_STILL_REASON_HEADER]: outcome.detail.slice(0, 500),
+			[CLIP_STILL_REASON_HEADER]: clipReasonHeaderValue(outcome.detail),
 			[CLIP_STILL_REASON_CODE_HEADER]: outcome.code
 		});
 		res.end();
@@ -152308,7 +152341,33 @@ function createClipStreamProducer(deps) {
 				}
 			});
 			activeByHost.set(hostKey, ahead + 1);
-			const run = (lanes.get(hostKey) ?? Promise.resolve()).then(() => runAttempts(input)).finally(() => {
+			let abandonedAt = null;
+			input.sink.onceClose(() => {
+				abandonedAt ??= Date.now();
+			});
+			const queuedAt = Date.now();
+			const run = (lanes.get(hostKey) ?? Promise.resolve()).then(async () => {
+				if (abandonedAt !== null) {
+					const waitedMs = abandonedAt - queuedAt;
+					deps.logger.info("videoclips: a queued clip stream was dropped, its peer had gone", {
+						tags: { deviceId },
+						meta: {
+							clipKey: input.row.clipKey,
+							twin: input.twin,
+							branch: "queued-peer-gone",
+							waitedMs,
+							aheadWhenQueued: ahead
+						}
+					});
+					return {
+						kind: "refused",
+						code: "fetch-failed",
+						detail: `the peer left after ${String(waitedMs)} ms in the queue, so this clip was never fetched`,
+						attempts: 0
+					};
+				}
+				return await runAttempts(input);
+			}).finally(() => {
 				const left = (activeByHost.get(hostKey) ?? 1) - 1;
 				if (left <= 0) activeByHost.delete(hostKey);
 				else activeByHost.set(hostKey, left);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@camstack/addon-provider-reolink",
-  "version": "1.2.161",
+  "version": "1.2.162",
   "description": "Reolink camera device provider addon for CamStack — native Baichuan protocol",
   "keywords": [
     "camstack",