TASK-0019: Bound NetGuard reality reporting after core polls - #9
Merged
lr00rl merged 1 commit intoAug 4, 2026
Merged
Conversation
Wire the existing fail-whole collector to the accepted agent endpoint behind an explicit local opt-in. Run the report after core poll work and share a ten-second deadline across collection and POST so degraded discovery or transport remains visible without delaying task, monitor, and log-source service in the current cycle. Constraint: Existing bearer authentication and server payload contracts must remain unchanged Constraint: Partial collector output must never be reported as node ground truth Rejected: Fire-and-forget reporting | overlapping host discovery would lose per-cycle ordering and error ownership Rejected: Reuse the shared 30-second request ceiling synchronously | optional reporting could starve core poll work Confidence: high Scope-risk: moderate Directive: Keep guard reality low-trust and fail-whole; do not move it before core poll operations without re-evaluating latency Tested: Focused normal and race tests; next-cycle recovery and deadline tests; gopls; release-workflow check; install-integrity check; go vet ./...; go test -race -cover ./...; allowed-path and redaction scans Not-tested: Live ss/ip/nft discovery and a cross-process agent-to-server run were intentionally not executed
Contributor
Author
|
Landed on |
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
1 participant
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.
Result
Verification
Boundaries
No SDK/proto, server, auth timing, workflow, install/release, signing, nft mutation, or deployment changes. Live host discovery and cross-process agent-to-server execution were intentionally not run.