Skip to content

Read the llms discovery link from <head> only - #8

Merged
zordhalo merged 1 commit into
mainfrom
fix/head-only-discovery
Oct 29, 2026
Merged

zordhalo merged 1 commit into
mainfrom
fix/head-only-discovery

Conversation

@zordhalo

Copy link
Copy Markdown
Contributor

Follow-up to #7. The reader now fetches with same-origin cookies, so the URL it fetches must not be choosable by markup that page content can inject.

  • linkedHref() now reads link[rel="llms"] from document.head only; a <link> in the body is ignored.
  • A test pins it.
  • The README install step now says to put the link in <head>. Every documented framework snippet already does.

Raised by an automated security review of #7. The exposure was narrow: it was same-origin only, needed a click, and was shown to the same visitor. The fix is cheap, though.

🤖 Generated with Claude Code

Follow-up to #7. The reader now fetches with same-origin cookies, so the URL it fetches must
not be choosable by markup page content can inject: a <link rel="llms"> in the body is
ignored. Every documented install already puts it in <head>; the README now says so.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@zordhalo
zordhalo merged commit 29fb006 into main Oct 29, 2026
2 checks passed
@zordhalo
zordhalo deleted the fix/head-only-discovery branch October 29, 2026 21:48
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