Skip to content

fix: escape dynamic radar data in browser UI - #1

Open
DorianChn wants to merge 1 commit into
mainfrom
codex/harden-star-data-rendering-20260914
Open

DorianChn wants to merge 1 commit into
mainfrom
codex/harden-star-data-rendering-20260914

Conversation

@DorianChn

Copy link
Copy Markdown
Owner

Summary

  • escape GitHub-sourced star names, descriptions, tags, scores, and search highlights before inserting them into HTML
  • allow only http/https external links and add rel="noopener noreferrer" to new-tab links
  • encode project card JSON with encodeURIComponent instead of embedding raw JSON in an HTML attribute
  • validate the embedded browser JavaScript in CI

Validation

  • embedded JavaScript parsed with Node vm.Script
  • scripts/update_stars.py passed py_compile
  • manifest.json parsed successfully
  • git diff --check

This branch has not been deployed

No deployments
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