fix: read a member's descriptor from the page's own rel=openwebring link - #201
Merged
Conversation
A site that is a path on a shared host (a blog under /~name/) cannot put a file at that host's /.well-known/, which is why the spec's second discovery route is a <link rel="openwebring" href="..."> on the page. The check only tried the origin, so the first active member of any ring (Chovy's Blog, which links to the Profullstack ring and points at its own openwebring.json) came back with made_by unstated. The origin's well-known file is still tried first; when it yields no descriptor, the page's pointer is followed, resolved against the page, attribute order not assumed. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01XYae2mH3khdwiXUVzcVMDw
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Read a member's descriptor from the page's own rel=openwebring link
The check fetched only
<origin>/.well-known/openwebring.json, so a member whose site is a path on a shared host (Chovy's Blog atdev.profullstack.com/~anthony/blog/, the first active member of any ring) was marked active from its page links butmade_bystayed unstated although the page carries<link rel="openwebring" href="openwebring.json">and the file saysboth/ai-assisted.descriptorLinkFrom(html, pageUrl)finds that pointer (any rel list containingopenwebring, attribute order free, href resolved against the page) andcheckRingMemberfollows it when the origin yields no descriptor. Test added; web and ingest suites pass.🤖 Generated with Claude Code
https://claude.ai/code/session_01XYae2mH3khdwiXUVzcVMDw