Improve DNS output: distinguish NS resolution failures from domain errors #20

Merged
multica-agent merged 2 commits from agent/go-expert-developer/ed72bea3 into main 2026-06-08 05:59:35 +00:00
Showing only changes of commit b481905848 - Show all commits
+5 -9
View File
@@ -232,11 +232,11 @@ func (t *Traverser) processReferral(ctx context.Context, ref *Referral, cache *I
ref.State = StateResolved ref.State = StateResolved
} else { } else {
addrs, err := t.ResolveNS(ctx, ref.Name, cache, visitedCopy, t.depth) addrs, err := t.ResolveNS(ctx, ref.Name, cache, visitedCopy, t.depth)
nsName := strings.TrimSuffix(ref.Bailiwick, ".")
if nsName == "" || nsName == "." {
nsName = strings.TrimSuffix(ref.Name, ".")
}
if err != nil { if err != nil {
nsName := strings.TrimSuffix(ref.Bailiwick, ".")
if nsName == "" || nsName == "." {
nsName = strings.TrimSuffix(ref.Name, ".")
}
return &Response{ return &Response{
Referral: ref, Referral: ref,
Type: RespNSResolutionFailed, Type: RespNSResolutionFailed,
@@ -247,10 +247,6 @@ func (t *Traverser) processReferral(ctx context.Context, ref *Referral, cache *I
ref.Addresses = addrs ref.Addresses = addrs
ref.State = StateResolved ref.State = StateResolved
} else { } else {
nsName := strings.TrimSuffix(ref.Bailiwick, ".")
if nsName == "" || nsName == "." {
nsName = strings.TrimSuffix(ref.Name, ".")
}
return &Response{ return &Response{
Referral: ref, Referral: ref,
Type: RespNSResolutionFailed, Type: RespNSResolutionFailed,
@@ -371,7 +367,7 @@ func (t *Traverser) ResolveNS(ctx context.Context, nsName string, cache *InfoCac
break break
} }
if resp.Type == RespSERVFAIL || resp.Type == RespError { if resp.Type == RespSERVFAIL || resp.Type == RespError || resp.Type == RespNSResolutionFailed {
lastErr = fmt.Errorf("server error resolving %s: %s", nsName, resp.Type) lastErr = fmt.Errorf("server error resolving %s: %s", nsName, resp.Type)
continue continue
} }