Fix to correctly handle query TTL (#187)

G-Research · Dec 12, 2022 · 0c123f1 · 0c123f1
1 parent 16884c4
commit 0c123f1
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 0 deletions.
diff --git a/Consul.Test/PreparedQueryTest.cs b/Consul.Test/PreparedQueryTest.cs
@@ -17,6 +17,7 @@
 //  </copyright>
 // -----------------------------------------------------------------------
 
+using System;
 using System.Linq;
 using System.Threading.Tasks;
 using Xunit;
@@ -59,6 +60,10 @@ public async Task PreparedQuery_Test()
                     Service = "sql",
                     Near = "_agent",
                     OnlyPassing = true
+                },
+                DNS = new QueryDNSOptions
+                {
+                    TTL = TimeSpan.FromSeconds(5)
                 }
             };
 

diff --git a/Consul/PreparedQuery.cs b/Consul/PreparedQuery.cs
@@ -58,6 +58,7 @@ public class QueryDNSOptions
         /// <summary>
         /// TTL is the time to live for the served DNS results.
         /// </summary>
+        [JsonConverter(typeof(DurationTimespanConverter))]
         [JsonProperty(NullValueHandling = NullValueHandling.Ignore)]
         public TimeSpan? TTL { get; set; }
     }