Decode creation data when using Gelato's relay #2267
Conversation
| self.assertEqual( | ||
| result.initializer, | ||
| b"\xb6>\x80\r\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x01\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x01\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\xbd\x89\xa1\xceM\xde6\x8f\xfa\xb0\xec5Pn\xec\xe0\xb1\xff\xdcT\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x01@\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\xfd\x072\xdc\x9e0?\t\xfc\xef:s\x88\xad\x10\xa84Y\xec\x99\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00Z\xfez\x11\xe7\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x01\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x008\x19\xb8\x00\xc6{\xe6@)\xc19<\x8b.\r\rb}\xad\xe2\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00$\xfeQ\xf6C\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00)\xfc\xb4;FS\x1b\xca\x00=\xdc\x8f\xcbg\xff\xe9\x19\x00\xc7b\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00", | ||
| ) |
There was a problem hiding this comment.
This file test contains this long bytes, for me, it makes sense move them to another file to keep the tests more readable.
| @@ -0,0 +1,21 @@ | |||
| gelato_relay_1_balance_v2_abi = [ | |||
There was a problem hiding this comment.
Why in the init file? Would be better in a file called gelato_relay_abi.py? In the same file we could keep several versions.
| @@ -229,6 +233,15 @@ def _decode_creation_data(self, data: Union[bytes, str]) -> list[ProxyCreationDa | |||
| if not data: | |||
| return [] | |||
|
|
|||
| # Try to decode using Gelato Relayer | |||
| try: | |||
There was a problem hiding this comment.
Maybe I'm wrong but looks like the gelato decoding is not searching the creation data in the provide data.
I mean, it looks like a previous step to prepare the data to be decoded. I'd say that the gelato decoder could be outside of this function _decode_creation_data to prepare the data for it.
Anyway, no strong opinion but at least I would like to update the documentation of this function.
|
@moisses89 just applied your suggestions |
/creationendpoint #2263