Version check URL generated incorrectly; ensured no trailing '/' in proxy_path component

Author: tieneupin

src/murfey/client/update.py

@@ -15,9 +15,9 @@ def check(api_base: ParseResult, install: bool = True, force: bool = False):
     If the version number is outside the allowed range then this can trigger
     an update on the client, and in that case will terminate the process.
     """
-    proxy_path = api_base.path
+    proxy_path = api_base.path.rstrip("/")
     version_check_url = api_base._replace(
-        path=f"{proxy_path}/version", query=f"client_version={murfey.__version__}"
+        path=f"{proxy_path}/version/", query=f"client_version={murfey.__version__}"
     )
     server_reply = requests.get(version_check_url.geturl())
     if server_reply.status_code != 200:
@@ -60,7 +60,7 @@ def install_murfey(api_base: ParseResult, version: str) -> bool:
     Return 'true' on success and 'false' on error."""
 
     assert api_base.hostname is not None
-    proxy_path = api_base.path
+    proxy_path = api_base.path.rstrip("/")
     result = subprocess.run(
         [
             sys.executable,

src/murfey/client/websocket.py

@@ -27,7 +27,9 @@ def __init__(
         websocket.enableTrace(True)
 
         url = urllib.parse.urlparse(server)._replace(scheme="ws")
-        proxy_path = url.path  # Path component indicates what the proxy path used was
+        proxy_path = url.path.rstrip(
+            "/"
+        )  # Path component indicates what the proxy path used was
 
         self._address = url.geturl()
         self._alive = True