Merge pull request #113 from InjectiveLabs/f/fix_authz_msgexec

achilleas-kal · web-flow · commit 0db9a92d5939 · 2022-05-27T17:59:02.000+03:00
fix: MsgExec unpack method
diff --git a/examples/chain_client/20_MsgExec.py b/examples/chain_client/20_MsgExec.py
@@ -78,8 +78,13 @@ async def main() -> None:
         return
 
     sim_res_msg = ProtoMsgComposer.MsgResponses(sim_res.result.data, simulation=True)
-    print("simulation msg response")
     print(sim_res_msg)
+    unpacked_msg_res = ProtoMsgComposer.UnpackMsgExecResponse(
+        msg_type=msg0.__class__.__name__,
+        data=sim_res_msg[0]
+    )
+    print("simulation msg response")
+    print(unpacked_msg_res)
 
     # build tx
     gas_price = 500000000
diff --git a/pyinjective/composer.py b/pyinjective/composer.py
@@ -592,7 +592,6 @@ def UnpackMsgExecResponse(msg_type, data):
             "MsgSubaccountTransfer": injective_exchange_tx_pb.MsgSubaccountTransferResponse,
             "MsgLiquidatePosition": injective_exchange_tx_pb.MsgLiquidatePositionResponse,
             "MsgIncreasePositionMargin": injective_exchange_tx_pb.MsgIncreasePositionMarginResponse,
-            "MsgBid": injective_auction_tx_pb.MsgBidResponse,
         }
 
-        return header_map[msg_type].FromString(data)
+        return header_map[msg_type].FromString(bytes(data, "utf-8"))

Original file line number	Diff line number	Diff line change
`@@ -592,7 +592,6 @@ def UnpackMsgExecResponse(msg_type, data):`
`592`	`592`	`"MsgSubaccountTransfer": injective_exchange_tx_pb.MsgSubaccountTransferResponse,`
`593`	`593`	`"MsgLiquidatePosition": injective_exchange_tx_pb.MsgLiquidatePositionResponse,`
`594`	`594`	`"MsgIncreasePositionMargin": injective_exchange_tx_pb.MsgIncreasePositionMarginResponse,`
`595`		`- "MsgBid": injective_auction_tx_pb.MsgBidResponse,`
`596`	`595`	`}`
`597`	`596`
`598`		`- return header_map[msg_type].FromString(data)`
	`597`	`+ return header_map[msg_type].FromString(bytes(data, "utf-8"))`