From ed6edc7cb9d9ea5ab8148b7f61ab9d7588d99406 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 66f5d64df9..e1c64ff4cb 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 @@ -1223,7 +1223,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) {