Query Log Column Header Refinements

[jccairns]

Presently the Query Log column headers are:

• Time
• Client
• Domain
• Type
• Server
• Latency
• Status

From viewing the generated config file, it is evident that the TSV query log format is used by default. From the Query Logging section of example-dnscrypt-proxy.toml:
TSV format columns: timestamp, client_ip, query_name, query_type, return_code, duration, server, relay

It appears to me that presently the column labeled Server in the Query Log view is actually displaying the return_code data from the log file, and the column labeled Status is displaying the server from the log file. Additionally, there is no column corresponding to the relay from the log file.

[jccairns]

Proposed enhancement as a patch of 6d48638 commit (upload as attachment is failing):

diff --git a/files/usr/local/www/dnscrypt-proxy-querylog.php b/files/usr/local/www/dnscrypt-proxy-querylog.php
index 4b30774..004df70 100644
--- a/files/usr/local/www/dnscrypt-proxy-querylog.php
+++ b/files/usr/local/www/dnscrypt-proxy-querylog.php
@@ -157,9 +157,10 @@ display_top_tabs($tab_array);
 						<th><?=gettext("Client")?></th>
 						<th><?=gettext("Domain")?></th>
 						<th><?=gettext("Type")?></th>
-						<th><?=gettext("Server")?></th>
-						<th><?=gettext("Latency")?></th>
 						<th><?=gettext("Status")?></th>
+						<th><?=gettext("Latency")?></th>
+						<th><?=gettext("Server")?></th>
+                  <th><?=gettext("Relay")?></th>
 					</tr>
 				</thead>
 				<tbody>
@@ -189,9 +190,10 @@ if (file_exists($query_log_file) && is_readable($query_log_file)) {
 					'client' => $parts[1] ?? '',
 					'domain' => $parts[2] ?? '',
 					'type' => $parts[3] ?? '',
-					'server' => $parts[4] ?? '',
+					'status' => $parts[4] ?? '',
 					'latency' => $parts[5] ?? '',
-					'status' => $parts[6] ?? 'OK'
+					'server' => $parts[6] ?? 'OK',
+               'relay' => $parts[7] ?? 'N/A',
 				);
 
 				// Apply filters
@@ -232,9 +234,10 @@ if (empty($entries)): ?>
 						<td><?=htmlspecialchars($entry['client'])?></td>
 						<td><code><?=htmlspecialchars($entry['domain'])?></code></td>
 						<td><span class="label label-primary"><?=htmlspecialchars($entry['type'])?></span></td>
-						<td><?=htmlspecialchars($entry['server'])?></td>
+                  <td><span class="label label-<?=htmlspecialchars($status_class)?>"><?=htmlspecialchars($entry['status'])?></span></td>
 						<td><?=htmlspecialchars($entry['latency'])?></td>
-						<td><span class="label label-<?=htmlspecialchars($status_class)?>"><?=htmlspecialchars($entry['status'])?></span></td>
+						<td><?=htmlspecialchars($entry['server'])?></td>
+						<td><?=htmlspecialchars($entry['relay'])?></td>
 					</tr>
 <?php
 	endforeach;

[nopoz]

@jccairns Thanks for filing the issue. I've fixed this in PR #32 and I'll be creating a new release shortly.

If you haven't yet - please give the project a star if you like it.