fix(ci3): accept slashes in /list/<path:key> for merge-train history

Author: AztecBot

ci3/dashboard/rk.py

@@ -403,7 +403,11 @@ def show_section(section):
         follow='top'
     )
 
-@app.route('/list/<key>')
+# <path:key> (not <key>) so branch-qualified history keys like
+# `history_<hash>_merge-train/spartan` route correctly. WSGI decodes %2F to /
+# in PATH_INFO before Flask routes, so percent-encoding can't rescue a plain
+# <key> converter — only <path:key> matches segments containing /.
+@app.route('/list/<path:key>')
 @optional_auth
 def get_list(key):
     value = get_list_as_string(key)

ci3/exec_test

@@ -36,7 +36,7 @@ Env: REF_NAME=${REF_NAME:-} CURRENT_VERSION=${CURRENT_VERSION:-} CI_FULL=${CI_FU
 Date: $(date)
 System: ARCH=$(arch) CPUS=$(nproc --all) MEM=$(free -h | awk '/^Mem:/{print $2}') HOSTNAME=$(hostname)
 Resources: CPU_LIST=$CPU_LIST CPUS=$CPUS MEM=$MEM TIMEOUT=$TIMEOUT
-History: $DASHBOARD_URL/list/history_$test_hash${TARGET_BRANCH:+_${TARGET_BRANCH//\//_}}
+History: $DASHBOARD_URL/list/history_$test_hash${TARGET_BRANCH:+_$TARGET_BRANCH}
 
 EOF
 

ci3/run_test_cmd

@@ -236,10 +236,8 @@ function track_test {
   redis_cli LTRIM $list_key 0 $max_lines &>/dev/null
 }
 
-# Slashes in TARGET_BRANCH (e.g. merge-train/spartan) are replaced with underscores
-# so the key is safe to embed in the /list/<key> dashboard URL.
 function track_test_history {
-  track_test "history_${test_hash}${TARGET_BRANCH:+_${TARGET_BRANCH//\//_}}" "$1" ${2:-}
+  track_test "history_${test_hash}${TARGET_BRANCH:+_$TARGET_BRANCH}" "$1" ${2:-}
 }
 
 function track_test_failed {