From 34f4ec836d971e8009dbdb09a731dceb0f58519c Mon Sep 17 00:00:00 2001 From: nathan Date: Wed, 8 Nov 2017 11:38:22 +0100 Subject: [PATCH] code polish --- .../network/dns/resolver/DnsNameResolverContext.java | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/src/dorkbox/network/dns/resolver/DnsNameResolverContext.java b/src/dorkbox/network/dns/resolver/DnsNameResolverContext.java index c5e612fa..bae04eba 100644 --- a/src/dorkbox/network/dns/resolver/DnsNameResolverContext.java +++ b/src/dorkbox/network/dns/resolver/DnsNameResolverContext.java @@ -303,8 +303,7 @@ class DnsNameResolverContext { resolveQuery(nameServerAddrStream, nameServerAddrStreamIndex, question, - parent.dnsQueryLifecycleObserverFactory() - .newDnsQueryLifecycleObserver(question), + parent.dnsQueryLifecycleObserverFactory().newDnsQueryLifecycleObserver(question), promise); } @@ -387,6 +386,7 @@ class DnsNameResolverContext { final DnsResponse res = envelope.content(); final int code = res.getHeader() .getRcode(); + if (code == DnsResponseCode.NOERROR) { if (handleRedirect(question, envelope, queryLifecycleObserver, promise)) { // Was a redirect so return here as everything else is handled in handleRedirect(...) @@ -409,7 +409,11 @@ class DnsNameResolverContext { // Retry with the next server if the server did not tell us that the domain does not exist. if (code != DnsResponseCode.NXDOMAIN) { - resolveQuery(nameServerAddrStream, nameServerAddrStreamIndex + 1, question, queryLifecycleObserver.queryNoAnswer(code), promise); + resolveQuery(nameServerAddrStream, + nameServerAddrStreamIndex + 1, + question, + queryLifecycleObserver.queryNoAnswer(code), + promise); } else { queryLifecycleObserver.queryFailed(NXDOMAIN_QUERY_FAILED_EXCEPTION);