Skip to content

01a02ecf - Unlisted forwards belong to the upstream backend - #14

Open
TaprootFreak wants to merge 4 commits into
developfrom
01a02ecf-unknown-is-backend
Open

01a02ecf - Unlisted forwards belong to the upstream backend#14
TaprootFreak wants to merge 4 commits into
developfrom
01a02ecf-unknown-is-backend

Conversation

@TaprootFreak

@TaprootFreak TaprootFreak commented Aug 24, 2026

Copy link
Copy Markdown
Contributor

EN:
Unlisted traffic this process forwards is not an endpoint of this process; it belongs to the upstream HTTP backend.

DE:
Ungelisteter Traffic, den dieser Prozess weiterleitet, ist kein Endpoint dieses Prozesses; er gehoert zum Upstream-HTTP-Backend.

Details

Listed routes stay finished here and must never be forwarded. Unknown routes stay unnamed. Forwarding an unknown route does not make it a listed route.

test/test-server.sh pins the process bullet in CONTRIBUTING.md, both README sentences, and the REVIEW.md fail rule so that wording cannot drop while a weaker paragraph remains.

@TaprootFreak

Copy link
Copy Markdown
Contributor Author

EN: Ready after 4 review passes.
DE: Bereit nach 4 Review-Durchlaeufen.

Details

Pass 1: pin the REVIEW/CONTRIBUTING ownership sentences (they were missing).
Pass 2: lock complete single-line sentences instead of prefixes.
Pass 3: pin the full README lede instead of the bold suffix.
Pass 4: 0 findings.

Rejected, not counted: a block parser instead of the repo's existing one-line grep -Fq pins.

A route this process does not list and therefore forwards belongs to the
upstream HTTP backend. Forwarding it does not make it a listed route.
Grep pins now lock the process bullet, both README sentences, and
the REVIEW fail rule so unlisted forwards stay upstream endpoints.
Rewrap the CONTRIBUTING, README, and REVIEW ownership clauses onto
single lines so grep pins lock the complete wording, not a prefix.
The ownership grep now matches the complete unlisted-traffic sentence,
not only the bold suffix.
@TaprootFreak
TaprootFreak force-pushed the 01a02ecf-unknown-is-backend branch from f59517a to b3a6b90 Compare August 24, 2026 09:57
@TaprootFreak

Copy link
Copy Markdown
Contributor Author

EN: Ready after 5 review passes.
DE: Bereit nach 5 Review-Durchlaeufen.

Details

Pass 1: pin REVIEW/CONTRIBUTING ownership sentences.
Pass 2: lock complete single-line sentences instead of prefixes.
Pass 3: pin the full README lede instead of the bold suffix.
Pass 4: 0 findings on that wording.
Pass 5 (rebase onto GET / 302): 0 counted findings.

Rejected, not counted:

  • A block parser instead of the repo's one-line grep -Fq pins.
  • Pinning the CONTRIBUTING Testing paragraph as a second ownership sentence. That paragraph describes the pins; the binding rule is the process bullet, which is already uniquely pinned. A second pin there would match the summary again if the process bullet disappeared.

Rebased onto develop after #13 (GET / 302 to swagger). Combined README lede keeps both the 302 and the unlisted-ownership sentence.

@TaprootFreak
TaprootFreak marked this pull request as ready for review August 24, 2026 10:02
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