From 4b36a5b08b0ec09eddf61cd330aec2a86048a122 Mon Sep 17 00:00:00 2001 From: Norman Maurer Date: Mon, 15 Apr 2019 21:41:04 +0200 Subject: [PATCH] Correctly calculate ttl for AuthoritativeNameServer when update existing records (#9051) Motivation: We did not correctly calculate the new ttl as we did forget to add `this.` Modifications: Add .this and so correctly calculate the TTL Result: Use correct TTL for authoritative nameservers when updating these. --- .../src/main/java/io/netty/resolver/dns/DnsResolveContext.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/resolver-dns/src/main/java/io/netty/resolver/dns/DnsResolveContext.java b/resolver-dns/src/main/java/io/netty/resolver/dns/DnsResolveContext.java index 88fcdee70a..5c5a6291b0 100644 --- a/resolver-dns/src/main/java/io/netty/resolver/dns/DnsResolveContext.java +++ b/resolver-dns/src/main/java/io/netty/resolver/dns/DnsResolveContext.java @@ -1232,7 +1232,7 @@ abstract class DnsResolveContext { void update(InetSocketAddress address, long ttl) { assert this.address == null || this.address.isUnresolved(); this.address = address; - this.ttl = min(ttl, ttl); + this.ttl = min(this.ttl, ttl); } void update(InetSocketAddress address) {