Skip to content

Commit

Permalink
fix: wrong signature encoding
Browse files Browse the repository at this point in the history
  • Loading branch information
fubuloubu committed Jun 29, 2023
1 parent 3c6d3b8 commit 87ef9f0
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions ape_safe/client.py
Original file line number Diff line number Diff line change
Expand Up @@ -271,7 +271,7 @@ def post_transaction(self, safe_tx: SafeTx, sigs: Optional[List[MessageSignature
tx_data = UnexecutedTxData.from_safe_tx(safe_tx)
if sigs:
tx_data.signatures = HexBytes(
reduce(lambda raw_sig, next_sig: raw_sig + next_sig.encode_vrs(), sigs, b"")
reduce(lambda raw_sig, next_sig: raw_sig + next_sig.encode_rsv(), sigs, b"")
)

url = f"{self.transaction_service_url}/api/v1/multisig-transactions"
Expand All @@ -286,7 +286,7 @@ def post_signature(self, safe_tx_hash: SafeTxID, signature: MessageSignature):
f"/v1/multisig-transactions/{str(safe_tx_hash)}/confirmations"
)
response = requests.post(
url, json={"origin": "ApeWorX/ape-safe", "signature": signature.encode_vrs().hex()}
url, json={"origin": "ApeWorX/ape-safe", "signature": signature.encode_rsv().hex()}
)

if not response.ok:
Expand Down

0 comments on commit 87ef9f0

Please sign in to comment.