-
Notifications
You must be signed in to change notification settings - Fork 1.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Events filtering not working with EIP checksum addresses (Ganache-CLI) #674
Comments
Can you expand on what "Not Working:" means. It's not clear to me what is actually broken/failing here. |
@pipermerriam What is meant there is that events filtering is not working if EIP checksum address is provided there. The filter always returns empty list on |
Sounds like an issue with Ganache since filtering is done on the backend and all that web3.py is doing is making RPC requests to get whatever entries the backend returns. |
@jordanjambazov could you solve this? Maybe by replacing ganache with something else. This is also a very important blocker for me. |
@EralpB Unfortunately currently I am using a forked monkey patched version of web3.py: jordanjambazov@a61b382 |
@EralpB @jordanjambazov can you help provide the ganache version issue and such on this issue: trufflesuite/ganache#494 ? |
@carver I commented on the version of Ganache-CLI, it was reproducible in v6.1.0-beta.2 |
@jordanjambazov are you getting the same problem when using the new |
If you're dealing with this error consider speaking up in the following github issue in the ganache codebase. |
Everyone who ran into this issue, please upgrade and try again. It's believed to be fixed! Your versions should be:
Please come back and comment if it is still not working as expected. |
I'm still having this issue with:
The following filter does not work:
but this does:
Where |
This was marked resolved in ganache-core@2.1.3 and ganache-cli@6.1.4 Can you upgrade and try again? |
@carver Working, thank you! |
Me I have |
Filtering events in ganache from web3 is broken. I submitted a fix and am waiting for it to get merged and included in ganache -> MetaMask/web3-provider-engine#280 |
What was wrong?
Maybe the issue is not exactly web3.py related, but when using EIP-55 checksum addresses event filtering is not working. My guess is that the reason is that Ganache-CLI is not producing proper addresses.
Event Filter done as follows:
Checking for events
Not Working:
Working (with monkey patched web3.utils.validation.validate_address):
The monkey patch is to simply ignore the EIP checksum validation.
How can it be fixed?
Not sure what is the best way to fix it. Simply ignoring checksum validation seems not right. I'm glad to discuss.
editor: Let's add an optional middleware that can be injected to the innermost layer that forces all hex to lowercase, similar to the
geth_poa_middleware
The text was updated successfully, but these errors were encountered: