From 4d9268d10453d25fd78a841fa56e7f515bfaa0cb Mon Sep 17 00:00:00 2001
From: Ivan Enderlin <ivan@mnt.io>
Date: Wed, 25 Feb 2026 15:36:42 +0100
Subject: [PATCH] fix(xtask): Fix an infinite loop when parsing the fields of a
 span.

The error was coming from the subslice from `message` instead of
`fields`.

This patch also fixes an HTML error (`</li>` right after `<ul>`).
---
 xtask/src/log/overview.rs | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/xtask/src/log/overview.rs b/xtask/src/log/overview.rs
index dea2d85dc..2f2547a54 100644
--- a/xtask/src/log/overview.rs
+++ b/xtask/src/log/overview.rs
@@ -72,7 +72,7 @@ lazy_static! {
             ^
 
             # A name.
-            (?<name>[\w\d_]+)
+            \s*(?<name>[\w\d_]+)
             # Equal
             =
             # A value, which can be anything: it stops when a new field is found.
@@ -333,9 +333,9 @@ pub(super) fn run(log_path: path::PathBuf, output_path: path::PathBuf) -> Result
             .unwrap();
 
             if let Some(fields) = captures.name("fields") {
-                writeln!(buffer, "<ul class=\"fields\"></li>").unwrap();
+                writeln!(buffer, "<ul class=\"fields\">").unwrap();
 
-                let mut fields = &message[fields.start()..];
+                let mut fields = &message[fields.start()..fields.end()];
 
                 while let Some(captures) = fields_parser.captures(fields) {
                     let name = captures
@@ -350,7 +350,7 @@ pub(super) fn run(log_path: path::PathBuf, output_path: path::PathBuf) -> Result
                     writeln!(buffer, "<li><span>{name}</span><span>{value}</span></li>").unwrap();
 
                     if let Some(next_fields) = captures.name("next_fields") {
-                        fields = &message[next_fields.start()..];
+                        fields = &fields[next_fields.start()..];
                     } else {
                         break;
                     }