Merge pull request #540 from OpenMined/timeout_environment

Added environment variable for client timeout
OpenMined · Feb 4, 2025 · b355c57 · b355c57
2 parents a35fddc + e529b49
commit b355c57
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/syftbox/client/base.py b/syftbox/client/base.py
@@ -131,7 +131,7 @@ def from_config(
             base_url=str(config.server_url),
             follow_redirects=True,
             headers=cls._make_headers(config),
-            timeout=10,
+            timeout=config.client_timeout,
             transport=transport,
         )
         return cls(conn)
diff --git a/syftbox/lib/client_config.py b/syftbox/lib/client_config.py
@@ -62,6 +62,9 @@ class SyftClientConfig(BaseModel):
     path: Path = Field(exclude=True, description="Path to the config file")
     """Path to the config file"""
 
+    client_timeout: float = Field(default=5, description="Timeout used by the client connection to the SyftBox server")
+    """Timeout used by the client connection to the SyftBox server"""
+
     @field_validator("client_url", mode="before")
     def port_to_url(cls, val: Union[int, str]) -> Optional[str]:
         if isinstance(val, int):