Skip to content

docs(trustless): clarify streaming gateway behavior & response headers#517

Merged
lidel merged 3 commits into
mainfrom
rvagg/trustless-update
Oct 18, 2025
Merged

docs(trustless): clarify streaming gateway behavior & response headers#517
lidel merged 3 commits into
mainfrom
rvagg/trustless-update

Conversation

@rvagg

@rvagg rvagg commented Oct 11, 2025

Copy link
Copy Markdown
Member
  • Add Accept vs format parameter precedence
  • Clarify 404 vs streaming behavior for missing blocks
  • Document Cache-Control: only-if-cached (412 response)
  • Add X-Ipfs-Roots guidance for streaming gateways
  • Document Etag implementation variance
  • Add Content-Disposition filename guidance (.car vs .bin)
  • Improve wording (CAR stream vs CAR file)

- Add Accept vs format parameter precedence
- Clarify 404 vs streaming behavior for missing blocks
- Document Cache-Control: only-if-cached (412 response)
- Add X-Ipfs-Roots guidance for streaming gateways
- Document Etag implementation variance
- Add Content-Disposition filename guidance (.car vs .bin)
- Improve wording (CAR stream vs CAR file)
@rvagg
rvagg requested a review from lidel as a code owner October 11, 2025 03:54
@github-actions

github-actions Bot commented Oct 11, 2025

Copy link
Copy Markdown

🚀 Build Preview on IPFS ready

@rvagg

rvagg commented Oct 11, 2025

Copy link
Copy Markdown
Member Author

For some background: the lens I'm using here is as someone who doesn't care much about the expansive definitions of "gateway" but only narrowly in the trustless portion of this, and only the pure CAR + raw block version as per the trustless spec. But because trustless says that it builds on the path spec, I have to care about the path spec but it contains a lot of extraneous information. So I've tried to bring local clarity into the trustless spec to highlight those things in path that are relevant. Plus, some additional clarifications I thought were warranted now I'm doing another pass on this and cleaning up frisbii to match.

lidel added 2 commits October 14, 2025 00:06
- move Cache-Control: only-if-cached to request headers section
- remove X-Ipfs-Path and X-Ipfs-Roots response headers section
- update editor affiliations and reorganize contributors
- update dates to 2025-10-13
specify that X-Ipfs-Path and X-Ipfs-Roots headers should be returned
with deserialized responses, and may be omitted with trustless response
types (raw blocks and CAR)

@lidel lidel left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you @rvagg lgtm, pushed small editorial changes, only change is to remove X-Ipfs-Path and X-Ipfs-Roots – see comment inline.

Lmk if these changes are acceptable.

Comment thread src/http-gateways/trustless-gateway.md Outdated
@rvagg

rvagg commented Oct 14, 2025

Copy link
Copy Markdown
Member Author

cool, lgtm

@lidel
lidel merged commit 110bf46 into main Oct 18, 2025
5 checks passed
@lidel
lidel deleted the rvagg/trustless-update branch October 18, 2025 04:20
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.

2 participants