tradingapi 0.3.1__tar.gz → 0.3.2__tar.gz

{tradingapi-0.3.1 → tradingapi-0.3.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: tradingapi
-Version: 0.3.1
+Version: 0.3.2
 Summary: Trade integration with brokers
 Author-email: Pankaj Sharma <sharma.pankaj.kumar@gmail.com>
 License-Expression: MIT

{tradingapi-0.3.1 → tradingapi-0.3.2}/pyproject.toml

@@ -28,7 +28,7 @@ packages = ["tradingapi"]
 
 [project]
 name = "tradingapi"
-version = "0.3.1"
+version = "0.3.2"
 description = "Trade integration with brokers"
 readme = "README.md"
 license = "MIT"

{tradingapi-0.3.1 → tradingapi-0.3.2}/tradingapi/fivepaisa.py

@@ -37,60 +37,51 @@ from py5paisa import FivePaisaClient
 
 
 def _patch_py5paisa_for_mom(api) -> None:
-    """Fix three upstream py5paisa bugs that break multi_order_Margin:
-
-      1. session is httpx.Client with 5s default timeout. Bump to 10s as a
-         safety margin (normal MOM responses are sub-second).
-      2. self.payload = GENERIC_PAYLOAD is a reference assignment, so each
-         call accumulates stale body keys across calls; the polluted body
-         makes the 5paisa server hang past 5s. Reset to a fresh deep copy
-         before each MOM call.
-      3. order_request() does res["body"]["Message"] for MOM, but the MOM
-         response has no "Message" key — KeyError is raised at the subscript,
-         swallowed by the bare except, and the function silently returns None.
-         Replace order_request for the MOM case to extract res["body"] directly
-         without touching "Message".
+    """Replace api.multi_order_Margin to bypass three upstream py5paisa bugs:
+
+      1. self.payload = GENERIC_PAYLOAD is a reference assignment shared across
+         every py5paisa method, so concurrent calls (status, balance, place_order)
+         clobber the MOM body between build and POST. Build payload locally.
+      2. order_request() does res["body"]["Message"] for MOM, but MOM responses
+         have no "Message" key — KeyError is swallowed by a bare except and the
+         function silently returns None. POST directly and read body.
+      3. Default 5s read timeout is too tight for SENSEX MOM under load.
+         Use a per-call 10s timeout (does not affect other API calls).
     """
     if getattr(api, "_tradingapi_patched", False):
         return
 
     import copy
-    from py5paisa.const import GENERIC_PAYLOAD
+    import httpx
+    from py5paisa.const import GENERIC_PAYLOAD, HEADERS
 
-    try:
-        api.session.timeout = 10
-    except Exception:
-        pass
+    # Dedicated session for MOM — never shares the pool with api.session,
+    # which other py5paisa calls (balance, order status, etc.) use concurrently.
+    mom_session = httpx.Client(verify=False, timeout=10)
 
-    original_order_request = api.order_request
+    def fixed_multi_order_Margin(**order):
+        payload = copy.deepcopy(GENERIC_PAYLOAD)
+        payload["body"]["ClientCode"] = api.client_code
+        payload["head"]["key"] = api.USER_KEY
+        for k, v in order.items():
+            payload["body"][k] = v
 
-    def safe_order_request(req_type):
-        if req_type != "MOM":
-            return original_order_request(req_type)
+        token = api.access_token or api.Jwt_token
+        headers = dict(HEADERS)
+        headers["Authorization"] = f"Bearer {token}"
+        headers["5Paisa-API-Uid"] = api.APIUID
         try:
-            api.payload["body"]["ClientCode"] = api.client_code
-            api.payload["head"]["key"] = api.USER_KEY
-            from py5paisa.const import HEADERS
-            token = api.access_token or api.Jwt_token
-            headers = dict(HEADERS)
-            headers["Authorization"] = f"Bearer {token}"
-            headers["5Paisa-API-Uid"] = api.APIUID
-            res = api.session.post(api.MULTIORDERMARGIN_ROUTE, json=api.payload, headers=headers).json()
-            api.payload = copy.deepcopy(GENERIC_PAYLOAD)
+            res = mom_session.post(
+                api.MULTIORDERMARGIN_ROUTE,
+                json=payload,
+                headers=headers,
+            ).json()
             return res.get("body")
         except Exception as e:
             from . import trading_logger
             trading_logger.log_warning("py5paisa MOM request failed", {"error": repr(e)})
-            api.payload = copy.deepcopy(GENERIC_PAYLOAD)
             return None
 
-    def fixed_multi_order_Margin(**order):
-        api.payload = copy.deepcopy(GENERIC_PAYLOAD)
-        for k, v in order.items():
-            api.payload["body"][k] = v
-        return safe_order_request("MOM")
-
-    api.order_request = safe_order_request
     api.multi_order_Margin = fixed_multi_order_Margin
     api._tradingapi_patched = True
 

{tradingapi-0.3.1 → tradingapi-0.3.2}/tradingapi.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: tradingapi
-Version: 0.3.1
+Version: 0.3.2
 Summary: Trade integration with brokers
 Author-email: Pankaj Sharma <sharma.pankaj.kumar@gmail.com>
 License-Expression: MIT