From ca1d43680880ac6e879df6a7b68f12cbcd508391 Mon Sep 17 00:00:00 2001
From: "Abdallah A. Zaqout" <26047413+zaqoutabed@users.noreply.github.com>
Date: Tue, 17 Mar 2026 03:57:36 +0300
Subject: [PATCH] Bench missing db_port attribute

Refactor database configuration properties to use getters.
---
 agent/bench.py | 15 ++++++++++++---
 1 file changed, 12 insertions(+), 3 deletions(-)

diff --git a/agent/bench.py b/agent/bench.py
index 9df6b483..92c1fdb3 100644
--- a/agent/bench.py
+++ b/agent/bench.py
@@ -55,9 +55,6 @@ def __init__(self, name: str, server: Server, mounts=None):
         self.apps_file = os.path.join(self.directory, "sites", "apps.txt")
         self.bench_config_file = os.path.join(self.directory, "config.json")
         self.config_file = os.path.join(self.directory, "sites", "common_site_config.json")
-        self.host = self.config.get("db_host", "localhost")
-        self.db_port = self.config.get("db_port", 3306)
-        self.docker_image = self.bench_config.get("docker_image")
         self.mounts = mounts
         if not (
             os.path.isdir(self.directory)
@@ -67,6 +64,18 @@ def __init__(self, name: str, server: Server, mounts=None):
         ):
             raise Exception
 
+    @property
+    def host(self):
+        return self.common_site_config.get("db_host", "localhost")
+
+    @property
+    def db_port(self):
+        return self.common_site_config.get("db_port", 3306)
+
+    @property
+    def docker_image(self):
+        return self.bench_config.get("docker_image")
+
     @step("Deploy Bench")
     def deploy(self):
         return self.start()