Commit 980223b
[fix]: bump handlebars to 4.7.9 (SECURE-3011) (#1437)
## Summary
Fixes [SECURE-3011](https://sendbird.atlassian.net/browse/SECURE-3011) —
bump `handlebars` from 4.7.8 → 4.7.9 to address
[CVE-2026-33941](https://nvd.nist.gov/vuln/detail/CVE-2026-33941) /
[GHSA-xjpj-3mr7-gcpf](GHSA-xjpj-3mr7-gcpf).
- **Severity**: High
- **Vulnerability**: Handlebars CLI precompiler allows code injection /
XSS via unescaped template filenames and CLI option values
(`--namespace`, `--commonjs`, `--handlebarPath`).
- **Reachability**: Conditionally Reachable (per Semgrep SSC).
## Impact analysis
- `handlebars` is a **transitive devDependency** via `plop@^2.5.3` →
`node-plop@0.26.3` → `handlebars@^4.4.3`.
- It is only used by the `generate-component` script (developer-only
component scaffolding) — **not bundled into the published package**.
- No runtime / consumer impact. Bumped for supply-chain hygiene.
## Approach
- `node-plop` still pins `handlebars@^4.4.3`, so a direct `yarn up`
wouldn't shift the resolution. Added a yarn `resolutions` entry to force
`4.7.9`:
```json
"handlebars@^4.4.3": "^4.7.9"
```
- `yarn.lock` now resolves `handlebars@npm:4.7.9`.
## Test plan
- [x] `yarn install` — resolution applied cleanly
(`handlebars@npm:4.7.9` in `yarn.lock`)
- [x] `yarn build` — exit 0, dist generated (warnings are pre-existing,
unrelated to this change)
- [x] `yarn test` — 941 passed / 10 skipped / 0 failed across 168 suites
🤖 Generated with [Claude Code](https://claude.com/claude-code)
[SECURE-3011]:
https://sendbird.atlassian.net/browse/SECURE-3011?atlOrigin=eyJpIjoiNWRkNTljNzYxNjVmNDY3MDlhMDU5Y2ZhYzA5YTRkZjUiLCJwIjoiZ2l0aHViLWNvbS1KU1cifQ
Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>1 parent 5f0125c commit 980223b
2 files changed
Lines changed: 6 additions & 5 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
170 | 170 | | |
171 | 171 | | |
172 | 172 | | |
173 | | - | |
| 173 | + | |
| 174 | + | |
174 | 175 | | |
175 | 176 | | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
8183 | 8183 | | |
8184 | 8184 | | |
8185 | 8185 | | |
8186 | | - | |
8187 | | - | |
8188 | | - | |
| 8186 | + | |
| 8187 | + | |
| 8188 | + | |
8189 | 8189 | | |
8190 | 8190 | | |
8191 | 8191 | | |
| |||
8197 | 8197 | | |
8198 | 8198 | | |
8199 | 8199 | | |
8200 | | - | |
| 8200 | + | |
8201 | 8201 | | |
8202 | 8202 | | |
8203 | 8203 | | |
| |||
0 commit comments