fix(platform): blob read URLs carry no query string; 0.26.2 - #30
Merged
Conversation
`walrusBlobReadUrl` returned `/v1/blobs/{id}?strict_consistency_check=false`
to pin the aggregator's read mode. That pin was a no-op and a liability:
- `false` has been Walrus's default since v1.37, so the bytes, length and
ETag are identical with or without it (checked against the testnet
aggregator).
- The aggregator deserialises read options with deny_unknown_fields, so
the pin's only failure mode is a rename that turns every cover URL
into a 400. Dropping it trades that outage for, at worst, a slower
read if the default ever flips back.
- A query string splits every cache into one entry per spelling of the
same blob: browser, edge, and the CDN's canonical transform keys.
Read policy belongs in the CDN that fronts the aggregator, server-side,
in one place. Cover URLs from the catalog reads are now the canonical
`/v1/blobs/{id}`.
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_0117boNS2uJCQr7L2n94xx53
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.
What
walrusBlobReadUrlnow returns the canonical/v1/blobs/{blob_id}with no query string, and@misofm/platformgoes to 0.26.2. Cover URLs from the catalog reads change accordingly.Why the
?strict_consistency_check=falsepin goesfalsehas been Walrus's default since v1.37 (November 2025). Bytes, length and ETag are identical with or without it on the testnet aggregator.deny_unknown_fields, so a future rename of the flag would turn every pinned cover URL into a 400. Without the pin the worst case is a slower read if the default ever flips back.Consumers
All pin
0.26.1exactly and need a bump onceplatform-v0.26.2is published:misofm/app,misofm/cli, and the four services inmisofm/api(auth, crank, platform, read). The app has one test fixture carrying the old query string.Verification
bun run typecheck,bun run test,bun run build,bun run test:consumerpass.codegen:checkwas not run here: it needs the sibling Move checkouts, and this change touches no generated code.🤖 Generated with Claude Code
https://claude.ai/code/session_0117boNS2uJCQr7L2n94xx53