From 2f1aff604a218ac28ac22fd5f60433f0d7aac10b Mon Sep 17 00:00:00 2001
From: RhodeCode Admin <admin@rhodecode.com>
Date: Wed, 3 Dec 2025 13:38:44 +0100
Subject: [PATCH] svn: fix ssh subcommand type with missing space

---
 rhodecode/apps/ssh_support/lib/backends/__init__.py | 13 ++++++++++---
 rhodecode/apps/ssh_support/lib/backends/svn.py      |  2 +-
 rhodecode/lib/svn_txn_utils.py                      | 12 ++++++++----
 3 files changed, 19 insertions(+), 8 deletions(-)

diff --git a/rhodecode/apps/ssh_support/lib/backends/__init__.py b/rhodecode/apps/ssh_support/lib/backends/__init__.py
index fd5837f7..de813e4e 100644
--- a/rhodecode/apps/ssh_support/lib/backends/__init__.py
+++ b/rhodecode/apps/ssh_support/lib/backends/__init__.py
@@ -234,15 +234,22 @@ class SshWrapper(object):
         scm_detected, scm_repo, scm_mode = self.get_repo_details(mode)
 
         log.debug(
-            'Mode: `%s` User: `name:%s : id:%s` Shell: `%s` SSH Command: `"%s"` '
-            "SCM_DETECTED: `%s` SCM Mode: `%s` SCM Repo: `%s`",
+            ""
+            "mode:`%s` "
+            "user_name:`%s`"
+            "user_id:`%s`"
+            "shell: `%s` "
+            'ssh_command:`"%s"` '
+            "scm_mode:`%s` "
+            "scm_type:`%s` "
+            "scm_repo_name:`%s`",
             mode,
             username,
             user_id,
             shell,
             self.command,
-            scm_detected,
             scm_mode,
+            scm_detected,
             scm_repo,
         )
 
diff --git a/rhodecode/apps/ssh_support/lib/backends/svn.py b/rhodecode/apps/ssh_support/lib/backends/svn.py
index 5c24be24..12403990 100644
--- a/rhodecode/apps/ssh_support/lib/backends/svn.py
+++ b/rhodecode/apps/ssh_support/lib/backends/svn.py
@@ -118,7 +118,7 @@ class SubversionTunnelWrapper(object):
         ra_client = self._svn_bytes(response["ra_client"])
         client = b" " + self._svn_bytes(response["client"]) or b""
 
-        buffer_ = b"( %b ( %b ) %b%b(%b) )" % (version, capabilities, url, ra_client, client)
+        buffer_ = b"( %b ( %b ) %b%b(%b) )%b" % (version, capabilities, url, ra_client, client, b" ")
         return buffer_
 
     def patch_first_client_response(self, response, **kwargs):
diff --git a/rhodecode/lib/svn_txn_utils.py b/rhodecode/lib/svn_txn_utils.py
index 24d6139b..1f4703b4 100644
--- a/rhodecode/lib/svn_txn_utils.py
+++ b/rhodecode/lib/svn_txn_utils.py
@@ -117,13 +117,17 @@ def get_txn_id_from_store(repo_path, svn_txn_id, rm_on_read=False):
     data = {}
     redis_conn.get(store_key)
     raw_data = "not-set"
+
     try:
         raw_data = redis_conn.get(store_key)
-        if not raw_data:
-            raise ValueError(f"Failed to get txn_id metadata, from store: {store_key}")
-        data = json.loads(raw_data)
+        if raw_data:
+            data = json.loads(raw_data)
     except Exception:
-        log.exception("Failed to get txn_id metadata: %s", raw_data)
+        log.exception("Failed to get txn_id metadata: redis_conn=%s raw_data=%s", redis_conn, raw_data)
+
+    if not data:
+        log.exception("Failed to get txn_id metadata: redis_conn=%s raw_data=%s", redis_conn, raw_data)
+        raise ValueError(f"Failed to get txn_id metadata, from store: {store_key}")
 
     if rm_on_read:
         log.debug("Cleaning up txn_id at %s", store_key)