Skip to content

docs: record the I12 refuse address as 127.0.0.1:1 - #128

Merged
plusky merged 1 commit into
mainfrom
docs/issue-127
Aug 19, 2026
Merged

docs: record the I12 refuse address as 127.0.0.1:1#128
plusky merged 1 commit into
mainfrom
docs/issue-127

Conversation

@plusky

@plusky plusky commented Aug 19, 2026

Copy link
Copy Markdown
Owner

PR #124 is already on main. This addresses #127, the leftover from that
PR: comments and DESIGN.md still (or in the commit message, still) told
the 127.0.0.2:1 / ENETUNREACH story, and cited a mutation-kill
(assert_ne!(ip, 127.0.0.1)) that is not in the tree.

The code uses 127.0.0.1:1. The load-bearing assertion is
assert!(addr.port() < 1024) — a bind-then-drop of an ephemeral port
fails the helper. Both refused_base_url copies and the Testing list
now say that. The red predecessor (a0f535f fails rust-macos) is
recorded as known history; main is not rewritten.

I12 is unchanged: no production code, no .without_url() change.

Adversarial review (pre-PR, two lenses)

  • DESIGN accuracy (MERGE-SAFE): current mechanism is 127.0.0.1:1
    plus port() < 1024; the only 127.0.0.2 mention is history of
    a0f535f. Follow-up nit: bind(:0) spelled as bind(127.0.0.1:0).
  • I12 evidence (MERGE-SAFE): production sanitization untouched;
    both helpers still carry port() < 1024; copies did not drift.

Verification: all five AGENTS.md commands green.

Closes #127

PR #124's first commit still describes 127.0.0.2:1 and a mutation-kill
that is not in the tree. The code uses 127.0.0.1:1; the load-bearing
assertion is port() < 1024. Comments and DESIGN.md now state that
mechanism. The red predecessor on main is recorded as known history
(#127); the commit graph is not rewritten.

Closes #127
@plusky plusky added the ai-assisted Authored or substantially written with an AI coding agent label Aug 19, 2026
@plusky
plusky merged commit 0cc22ff into main Aug 19, 2026
13 checks passed
@plusky
plusky deleted the docs/issue-127 branch August 19, 2026 06:39
@plusky plusky mentioned this pull request Aug 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ai-assisted Authored or substantially written with an AI coding agent

Projects

None yet

Development

Successfully merging this pull request may close these issues.

PR #124 landed a commit that fails rust-macos, and its message describes a mechanism the code does not use

1 participant