GitHub Advisoryadvisory · npm
GHSA-7q9x-8g6p-3x75 @grackle-ai/server: Unescaped Error String in renderPairingPage() HTML Template
LOW
Official page ↗
- Ecosystem
- npm
- Packages
- @grackle-ai/server
- Severity
- Low
- CVSS vector
- CVSS:4.0/AV:N/AC:L/AT:P/PR:N/UI:P/VC:N/VI:N/VA:N/SC:L/SI:L/SA:N
- Weaknesses
- CWE-79
- Reviewed
- 2026-03-25
Affected packages
• npm @grackle-ai/server: < 0.70.1 → fixed in 0.70.1
Description
Impact
The `renderPairingPage()` function embeds the `error` parameter directly into HTML without escaping:
```typescript
const errorHtml = error ? `${error}` : "";
```
All current call sites pass hardcoded strings, so this is not exploitable today. However, the function is architecturally fragile — if a future code change passes user-controlled or dynamic content into the error parameter, it would create an XSS vulnerability.
The `renderAuthorizePage()` function in the same file correctly uses `escapeHtml()` for dynamic content, making this an inconsistency.
Affected code:
• `packages/server/src/index.ts:64-89` — `renderPairingPage()` with unescaped error interpolation
• Compare: `packages/server/src/index.ts:130` — `renderAuthorizePage()` correctly uses `escapeHtml()`
Patches
v0.70.1
Fix: Apply `escapeHtml()` to the error parameter:
```typescript
const errorHtml = error ? `${escapeHtml(error)}` : "";
```
Workarounds
No workaround needed — all current callers pass hardcoded strings.
Resources
• CWE-79: Improper Neutralization of Input During Web Page Generation
• File: `packages/server/src/index.ts`
Related
Weakness · CWE (1)
- CWE-79 Improper Neutralization of Input During Web Page Generation weakness