security: bind special-purpose address decisions to reviewed IANA registry - #223
security: bind special-purpose address decisions to reviewed IANA registry#223seonghobae wants to merge 6 commits into
Conversation
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
@coderabbitai review Please review exact head |
|
Please perform a fresh exact-head semantic review of |
Rate Limit Exceeded
|
|
@cwl-noema-review After a qualifying exact-head OpenCode verdict, independently review |
seonghobae
left a comment
There was a problem hiding this comment.
Current-head source review for 811ba846757adb44dfc648b9b13761aa1536a110 found no blocking defect in the five-file special-purpose registry compatibility scope. The implementation applies reviewed most-specific IANA global exceptions before broader non-global compatibility ranges, recursively classifies IPv4-mapped IPv6 through the underlying IPv4 policy, preserves the narrower local-development authority path, performs no runtime registry lookup, and keeps ordinary stdlib classification for addresses outside the bounded overlay. The documented overlay matches the current IANA registries last updated 2025-10-09, including the 2001::/23 exception structure and current 192.0.0.0/24 exceptions. This is author-side review evidence only, not independent approval. Keep Draft until corrected central Dependency Review and authoritative Strix evidence run on the unchanged head and qualifying independent review exists.
|
@cwl-noema-review review Review exact current head |
Purpose
Reconstruct the unique special-purpose IP registry compatibility delta from stale/diverged Draft #173 directly on exact protected
mainwithout transferring historical checks, reviews, approvals, base identity, or mergeability.Python's
ipaddressclassification tables evolve with interpreter maintenance releases, while EgressWeave supports a Python version matrix and must keep destination authorization stable. The current IANA IPv4 and IPv6 Special-Purpose Address Space registries remain the authoritative reviewed source for the bounded compatibility overlay.Exact identity and test-first lineage
7faf7a3b8a47980113982914000e724ab6a6cda5;8312a000197eabd7d7d3f6341263c79f031;26be254deb11be6d5ff1bb10f35e9e70395b91be;811ba846757adb44dfc648b9b13761aa1536a110;74211307bf83d8089ca0c935a8a6664ca897ffce.Hosted RED run
31785513013failed on Python 3.10–3.14 exactly on the new compatibility contracts. The Python 3.14.6 lane reported 1,032 passing tests and 22 intended failures, demonstrating that the prior implementation depended on interpreter-versionis_private/is_globalmetadata.The final test/documentation correction accounts for
2001:2::/48being denied through the reviewed broader2001::/23implementation range rather than requiring a redundant source literal.Implemented boundary
2002::/16CPython compatibility decision, update procedure, and local-development non-widening contract.Fresh primary-source revalidation — 2026-08-29 KST
The live IANA IPv4 and IPv6 Special-Purpose Address Space registries still report Last Updated: 2025-10-09. The current exact-head overlay remains consistent with the reviewed version-sensitive subset: IPv4
192.0.0.0/24with globally reachable192.0.0.9/32and192.0.0.10/32exceptions; IPv664:ff9b:1::/48,100:0:0:1::/64,2001::/23,3fff::/20, and5f00::/16non-global handling; and the current globally reachable exceptions inside2001::/23.2002::/16remains a deliberate conservative EgressWeave/CPython compatibility policy rather than an overclaim that IANA marks it simplyGlobally Reachable=False.docs/research/special-purpose-address-classification.mdalready carries APA 7-style primary-source traceability and an explicit future-update procedure. Because the primary registries have not changed, no source/document blob is churned merely to update a retrieval date.This repository has no Storybook, Figma, or web design-system surface in the current product boundary; fresh repository searches returned no
storybook,figma, or design-system UI evidence, so UI/accessibility design work is not applicable to this security-policy slice.Exact-current-head verification
Exact head
811ba846757adb44dfc648b9b13761aa1536a110remains unchanged and live protectedmainremains7faf7a3b8a47980113982914000e724ab6a6cda5.31786475525: terminal success;31786475542: terminal success;31786475686: wrapper terminal success, but dependency-review job94723408440skipped the actual pinned Dependency review step; this is non-passing supply-chain evidence under the fail-closed acceptance contract;COMMENTEDreview only, not an independent approval;Live protection and central dependencies
Effective organization ruleset
18156473 — CWL Central required workflowsis active on the default branch and requires one approving review, stale-review dismissal on push, review-thread resolution, and the central OpenCode, scheduler, Security Scan, Strix, SAST, and Noema workflows. OrganizationAdmin bypass exists but is not merge authority for this lane and must not be used.The old Strix prerequisite
.github#965is closed unmerged and superseded. The current bounded Strix exact-artifact successor is.github#1009; it is still open and not merge-authoritative. The Dependency Review fail-closed owner remains.github#897; its current source-level repair is open and technically demonstrates the exact HTTP-200/pinned-action path, but its current required Strix provider evidence and independent approval are still non-passing. These central lanes are read-only dependencies from EgressWeave and must not be raced or locally duplicated.Merge contract
Keep Draft. Do not merge until:
ContextualWisdomLab/.github#897or a freshly verified protected successor) integrates and a fresh unchanged-head Security Scan actually executes and succeeds at the immutable-pinned Dependency Review step;ContextualWisdomLab/.github#1009or a freshly verified protected successor) integrates and this exact EgressWeave head receives authoritative, internally coherent Strix evidence;Pending, queued, skipped-required, stale, predecessor-head, synthetic, wrapper-only, author-only, model-only, or provider-unavailable evidence is non-passing. Do not self-approve or use administrator bypass.
Progresses #95. Supersedes #173; historical predecessor evidence is intentionally not transferred.