UI: Cap at timeout to handle system sleep/suspend

Author: jedisct1

dnscrypt-proxy/monitoring_ui.go

@@ -324,7 +324,19 @@ func (ui *MonitoringUI) UpdateMetrics(pluginsState PluginsState, msg *dns.Msg, s
 	}
 
 	// Update response time - back to counters lock
-	responseTime := time.Since(start).Milliseconds()
+	var responseTime int64
+	if !pluginsState.requestStart.IsZero() && !pluginsState.requestEnd.IsZero() {
+		responseTime = pluginsState.requestEnd.Sub(pluginsState.requestStart).Milliseconds()
+	} else {
+		// For incomplete queries, use timeout duration
+		responseTime = pluginsState.timeout.Milliseconds()
+	}
+
+	// Cap at timeout to handle system sleep/suspend
+	maxResponseTime := pluginsState.timeout.Milliseconds()
+	if responseTime > maxResponseTime {
+		responseTime = maxResponseTime
+	}
 	mc.countersMutex.Lock()
 	mc.responseTimeSum += uint64(responseTime)
 	mc.responseTimeCount++

dnscrypt-proxy/plugin_query_log.go

@@ -83,6 +83,14 @@ func (plugin *PluginQueryLog) Eval(pluginsState *PluginsState, msg *dns.Msg) err
 	var requestDuration time.Duration
 	if !pluginsState.requestStart.IsZero() && !pluginsState.requestEnd.IsZero() {
 		requestDuration = pluginsState.requestEnd.Sub(pluginsState.requestStart)
+	} else {
+		// For incomplete queries, use timeout duration
+		requestDuration = pluginsState.timeout
+	}
+
+	// Cap at timeout to handle system sleep/suspend
+	if requestDuration > pluginsState.timeout {
+		requestDuration = pluginsState.timeout
 	}
 	var line string
 	if plugin.format == "tsv" {