Skip to content

fix: recover safely from malformed search fragments - #6

Merged
siddath merged 1 commit into
mainfrom
codex/security-search-fragment
Oct 5, 2026
Merged

siddath merged 1 commit into
mainfrom
codex/security-search-fragment

Conversation

@siddath

@siddath siddath commented Oct 5, 2026

Copy link
Copy Markdown
Owner

A chapter URL such as #find-% or #find-%GG could throw during React rendering and blank the reader. Search fragments are now decoded once through a guarded helper; malformed encodings safely fall back to the first section, while valid Unicode searches still select matching commentary.

Addresses Codex Security Cloud finding csf_8bdfd647c3204be095674eac at 41c6d1d.

Validation: production build and 11 unit/compiler tests pass. All 14 existing browser cases pass; the four new malformed-fragment/direct/history/Unicode cases pass using installed Chrome. The new tests' initially ambiguous Original text locator was corrected and those cases rerun. npm install audit reports zero vulnerabilities on 5 October 2026. No new dependencies.

No merge, deployment, or cloud-finding closure performed. The fix handles the identified input boundary; a general application error boundary is outside this focused patch.

@vercel

vercel Bot commented Oct 5, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
ithaca-odyssey Ready Ready Preview Oct 5, 2026 9:45am UTC

@siddath
siddath marked this pull request as ready for review October 5, 2026 12:55
@siddath
siddath merged commit 83573ab into main Oct 5, 2026
4 checks passed
@siddath
siddath deleted the codex/security-search-fragment branch October 5, 2026 12:55

This branch was successfully deployed

1 active deployment
Preview — 2750eb25 Deployed Oct 5, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant