Compare commits
1 Commits
| Author | SHA1 | Date | |
|---|---|---|---|
| 10ee988ce2 |
@@ -1,4 +0,0 @@
|
|||||||
## 2025-06-03 - Avoid TextEncoder for string byte length
|
|
||||||
|
|
||||||
**Learning:** `new TextEncoder().encode(text).byteLength` allocates a `Uint8Array` for the entire string, which causes significant memory allocations and is ~100x slower for large JSON payloads in Bun/Node.
|
|
||||||
**Action:** Use `Buffer.byteLength(text)` when available (with a fallback to `TextEncoder` for browser compatibility) to compute string byte lengths without memory allocation overhead.
|
|
||||||
@@ -0,0 +1,4 @@
|
|||||||
|
## 2025-02-28 - Caddy HTTP Security Headers Configuration
|
||||||
|
**Vulnerability:** The application was missing basic security headers (X-Frame-Options, X-Content-Type-Options, Referrer-Policy), increasing exposure to clickjacking and MIME-sniffing attacks.
|
||||||
|
**Learning:** Security posture isn't just about application-level code. The reverse proxy (Caddy) configuration serves as the first line of defense and is the correct layer to enforce global HTTP response headers.
|
||||||
|
**Prevention:** Always verify that the reverse proxy layer enforces foundational HTTP security headers instead of relying solely on the application code to handle response security.
|
||||||
@@ -6,6 +6,12 @@
|
|||||||
:80 {
|
:80 {
|
||||||
encode zstd gzip
|
encode zstd gzip
|
||||||
|
|
||||||
|
header {
|
||||||
|
X-Frame-Options "DENY"
|
||||||
|
X-Content-Type-Options "nosniff"
|
||||||
|
Referrer-Policy "strict-origin-when-cross-origin"
|
||||||
|
}
|
||||||
|
|
||||||
handle /mcp {
|
handle /mcp {
|
||||||
reverse_proxy 127.0.0.1:3001
|
reverse_proxy 127.0.0.1:3001
|
||||||
}
|
}
|
||||||
|
|||||||
+1
-8
@@ -75,14 +75,7 @@ export function normalizeScene(input: unknown): ScenePayload {
|
|||||||
}
|
}
|
||||||
|
|
||||||
export function parseJsonBody<T = unknown>(text: string): T {
|
export function parseJsonBody<T = unknown>(text: string): T {
|
||||||
// Optimization: TextEncoder.encode creates a new Uint8Array and allocates memory for the entire
|
if (new TextEncoder().encode(text).byteLength > MAX_SCENE_BYTES) {
|
||||||
// string, which is slow for large JSON payloads. Buffer.byteLength counts bytes without allocation.
|
|
||||||
const byteLength =
|
|
||||||
typeof Buffer !== "undefined"
|
|
||||||
? Buffer.byteLength(text)
|
|
||||||
: new TextEncoder().encode(text).byteLength;
|
|
||||||
|
|
||||||
if (byteLength > MAX_SCENE_BYTES) {
|
|
||||||
throw new HttpError(413, "Payload too large");
|
throw new HttpError(413, "Payload too large");
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user