Repository navigation
[CC5-s1375d0] docs(txe): TXE-5670 the design doc says what an edge proof showed - #216
Merged
Merged
Conversation
…oof showed HTTPS-CHANNEL.md still said no Cloudflare path had been tried. It now says what the synthetic proof through a Cloudflare Tunnel showed, in generic terms: unary calls and refusals, held polls, idle gaps, a long stream, a 16 MiB request, a connector restart and a stopped origin; small request bodies held until the request ends, large ones streamed; HTTP/2 to the edge. It says what was not run and why. No host, zone setting, access configuration or reachability fact is in it. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
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
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.
Summary
txe/worker/HTTPS-CHANNEL.mdstill said no Cloudflare path had been tried. A synthetic proof has since run through a Cloudflare Tunnel, with the real channel handler behind it. A new section, "Through an edge", says what that showed, in generic terms:Unavailable;Unavailable.It also records two edge behaviours:
What was not run is stated, with why: the at-limit and over-limit requests could not fit the stream limit at the link's rate, so they were skipped, not passed. The section points to the fixture's lowered cap (#215) that lets them fit.
The status line and the open questions are updated to match.
Kept out: hostnames, zone settings, access configuration, security posture and reachability. This is the fixture and the protocol, not the production uploader, a real hub, a real identity provider or a real worker, and the doc says so.
Tests
Docs only.
Closes TXE-5670
🤖 Generated with Claude Code