fix(Client): wrong propose and delegate endpoints (#79)

* fix(Client): allow `**kwargs` pass-through for GET requests

* fix(Client): use v2 API for delegates

* fix(Client): updated URLs for new API

Author: fubuloubu

ape_safe/client/__init__.py

@@ -134,7 +134,7 @@ def _all_transactions(self) -> Iterator[SafeApiTxData]:
             url = data.get("next")
 
     def get_confirmations(self, safe_tx_hash: SafeTxID) -> Iterator[SafeTxConfirmation]:
-        response = self._get(f"/multisig-transactions/{safe_tx_hash}/raw")
+        response = self._get(f"/multisig-transactions/{safe_tx_hash}", api_version="v2")
         data = response.json()
         yield from map(SafeTxConfirmation.model_validate, data.get("confirmations", []))
 
@@ -169,8 +169,8 @@ def post_transaction(
             # Signature handled above.
             post_dict.pop("signatures")
 
-        url = f"/transactions/{tx_data.safe}/propose"
-        response = self._post(url, json=post_dict)
+        url = f"/safes/{tx_data.safe}/multisig-transactions"
+        response = self._post(url, json=post_dict, api_version="v2")
         return response
 
     def post_signatures(
@@ -185,7 +185,7 @@ def post_signatures(
             safe_tx_hash = safe_tx_or_hash
 
         safe_tx_hash = cast(SafeTxID, to_hex(HexBytes(safe_tx_hash)))
-        url = f"/transactions/{safe_tx_hash}/confirmations"
+        url = f"/multisig-transactions/{safe_tx_hash}/confirmations"
         signature = to_hex(
             HexBytes(b"".join([x.encode_rsv() for x in order_by_signer(signatures)]))
         )
@@ -216,7 +216,7 @@ def get_delegates(self) -> dict["AddressType", list["AddressType"]]:
         delegates: dict[AddressType, list[AddressType]] = {}
 
         while url:
-            response = self._get(url, params={"safe": self.address})
+            response = self._get(url, params={"safe": self.address}, api_version="v2")
             data = response.json()
 
             for delegate_info in map(DelegateInfo.model_validate, data.get("results", [])):
@@ -243,7 +243,7 @@ def add_delegate(self, delegate: "AddressType", label: str, delegator: "AccountA
             "label": label,
             "signature": sig.encode_rsv().hex(),
         }
-        self._post("/delegates", json=payload)
+        self._post("/delegates", json=payload, api_version="v2")
 
     def remove_delegate(self, delegate: "AddressType", delegator: "AccountAPI"):
         msg_hash = self.create_delegate_message(delegate)
@@ -256,7 +256,7 @@ def remove_delegate(self, delegate: "AddressType", delegator: "AccountAPI"):
             "delegator": delegator.address,
             "signature": sig.encode_rsv().hex(),
         }
-        self._delete(f"/delegates/{delegate}", json=payload)
+        self._delete(f"/delegates/{delegate}", json=payload, api_version="v2")
 
 
 __all__ = [

ape_safe/client/base.py

@@ -146,8 +146,8 @@ def session(self) -> requests.Session:
         session.mount("https://", adapter)
         return session
 
-    def _get(self, url: str, params: Optional[dict] = None) -> "Response":
-        return self._request("GET", url, params=params)
+    def _get(self, url: str, params: Optional[dict] = None, **kwargs) -> "Response":
+        return self._request("GET", url, params=params, **kwargs)
 
     def _post(self, url: str, json: Optional[dict] = None, **kwargs) -> "Response":
         return self._request("POST", url, json=json, **kwargs)