Skip to content

Attest retained AWG and VLESS services under leased locks - #166

Merged
Tim55667757 merged 1 commit into
developfrom
feature/128-service-attestation
Nov 7, 2026
Merged

Tim55667757 merged 1 commit into
developfrom
feature/128-service-attestation

Conversation

@Tim55667757

@Tim55667757 Tim55667757 commented Nov 7, 2026 •

Copy link
Copy Markdown
Contributor

Native WireGuard completion must preserve the actual unrelated AWG/VLESS services, rather than accepting source hashes supplied by a request. Add a concrete private observer that binds complete retained server bundles before I/O, acquires existing AWG then VLESS shared locks after common/WG coordination, and retains file, enable-link and process identities through two live observations and immutable completion publication.

AWG verifies credentials, peers, configured parameters, address/MTU/forwarding and the complete native nft policy against the retained firewall compiled in a fresh transient network namespace. VLESS verifies owned runtime/upgrade receipts, exact daemon PID/executable/arguments/cgroup and its actual systemd loaded credential, including the dynamic-UID read ACL, plus ownership of the TCP listener. Source-bound absence requires existing coordination and complete native namespace/listener absence; unloaded unit overrides and unexplained entries refuse. Legacy owner algorithms, installers, keys and service files remain unchanged. Historical completion replay skips live leases.

Continues #128; does not close it. These are configuration/service preservation probes; no traffic or YouTube acceptance is claimed. Protocol RPC, controller and public CLI integration remain separate work. No desktop VPN, WSL, production guest/cloud or local network operations were performed.

Validation on exact head c1cea21:

Added 71 deterministic cases, including 45 portable source/AWG/credential-ACL codec checks, actual POSIX ownership/proc/loaded-credential/lease drift cases, exact observer lifetime checks, and real retained AWG nft echo/list conformance in an ephemeral namespace. All new POSIX cases ran in Linux CI. Canonical ADR/engineering and reviewed RU/ZH catalogs updated; upstream decisions link to pinned primary sources. Independent exact-head AI static review found no remaining actionable issues; see its review COMMENT.

@Tim55667757 Tim55667757 added this to the release-2.1 milestone Nov 7, 2026 — with ChatGPT Codex Connector
@Tim55667757 Tim55667757 added the feature New complex feature or request label Nov 7, 2026 — with ChatGPT Codex Connector
@Tim55667757 Tim55667757 self-assigned this Nov 7, 2026

@Tim55667757 Tim55667757 left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Independent AI static review completed for exact head c1cea21 against 0548e74. All reported findings are resolved: compare live nft to the exact retained firewall compiled in an isolated namespace; reject unconfigured AWG timing/security overrides; verify the daemon's actual loaded systemd credential, process identity and PID-attributed listener; reject unloaded owner-unit drop-ins. Legacy AWG/VLESS owner derivations are preserved. The reviewer found no remaining actionable findings, and EN/RU/ZH semantics plus all four provenance pairs agree. Static review did not execute tests; approval remains conditional on all three exact-head CI gates succeeding. This COMMENT records independent AI review and does not impersonate a human APPROVE. Configuration/service probes do not establish client traffic or media playback health.

@Tim55667757
Tim55667757 marked this pull request as ready for review November 7, 2026 10:43

Copy link
Copy Markdown
Contributor Author

Reviewed

@Tim55667757
Tim55667757 merged commit 3184301 into develop Nov 7, 2026
4 checks passed
@Tim55667757
Tim55667757 deleted the feature/128-service-attestation branch November 7, 2026 10:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature New complex feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant