Add specific circuit failure error codes. Fixes #3717, #3543, #3364, #2888, #2859, #1580#3792
Open
Add specific circuit failure error codes. Fixes #3717, #3543, #3364, #2888, #2859, #1580#3792
Conversation
…2888, #2859, #1580 - adds ErrorType constants for rejected-by-application, DNS resolution failed, port not allowed, invalid link destination, and resources not available - adds corresponding CircuitFailureCause strings reported in circuit events - extracts classifyDialError() in route handler to map dial errors to specific error codes using typed errors, syscall constants, and string matching - detects DNS errors via *net.DNSError and string fallback for ER/T hosted services where errors are serialized through the SDK message protocol - detects resource exhaustion via EMFILE, ENFILE, ENOBUFS syscall errors - introduces InvalidLinkDestinationError typed error in forwarder package - adds unit tests covering all 16 classification cases - adds integration tests for rejected-by-application (SDK host), DNS resolution failed, connection refused, and port not allowed (ER/T host mode) with circuit event verification - adds CreateEnrollAndStartTunnelerEdgeRouterWithCfgTweaks to test context
plorenz
force-pushed
the
specific-router-error-types
branch
from
6f7f18c to
9fb5773
Compare
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.
port not allowed, invalid link destination, and resources not available
error codes using typed errors, syscall constants, and string matching
services where errors are serialized through the SDK message protocol
DNS resolution failed, connection refused, and port not allowed
(ER/T host mode) with circuit event verification