fix: client banner to obtain SI version from SmartInspect

Author: u-chaCP

protocols/tcp_protocol.py

@@ -12,13 +12,12 @@
 
 
 class TcpProtocol(Protocol):
-    __BUFFER_SIZE = 0x2000
-    __CLIENT_BANNER = bytearray(f"SmartInspect Python Library v\n", encoding="UTF-8")
-    # __CLIENT_BANNER = bytearray(f"SmartInspect Python Library v{SmartInspect.get_version()}\n", encoding="UTF-8")
-    __ANSWER_BUFFER_SIZE = 0x2000
-    _hostname = "127.0.0.1"
-    _timeout = 30000
-    _port = 4228
+    __BUFFER_SIZE: int = 0x2000
+    __CLIENT_BANNER_TEMPLATE: str = "SmartInspect Python Library v{}\n"
+    __ANSWER_BUFFER_SIZE: int = 0x2000
+    _hostname: str = "127.0.0.1"
+    _timeout: int = 30000
+    _port: int = 4228
 
     def __init__(self):
         super().__init__()
@@ -59,7 +58,9 @@ def _read_server_banner(self) -> None:
                                     "Connection has been closed unexpectedly")
 
     def _send_client_banner(self) -> None:
-        self.__stream.write(self.__CLIENT_BANNER)
+        from smartinspect import SmartInspect
+        si_version = SmartInspect.get_version()
+        self.__stream.write(self.__CLIENT_BANNER_TEMPLATE.format(si_version).encode("UTF-8"))
         self.__stream.flush()
 
     def _internal_connect(self):

smartinspect.py

@@ -68,9 +68,9 @@ def set_resolution(self, resolution: ClockResolution) -> None:
         if isinstance(resolution, ClockResolution):
             self.__resolution = resolution
 
-    @property
-    def version(self) -> str:
-        return self.__VERSION
+    @classmethod
+    def get_version(cls) -> str:
+        return cls.__VERSION
 
     @property
     def hostname(self) -> str: