search

Consensys / ethql

A GraphQL interface to Ethereum :fire:
Apache License 2.0
622 stars 85 forks source link

Issue 118: Topic Filter in Logs (Rebased) #121

Closed kshinn closed 5 years ago

kshinn commented 5 years ago

As described in #118 when submitting queries that have the log(filter: {topics: []}) filter engaged, all logs get returned no matter what is in the topic filter. I found that the topic argument was not getting passed into getPastLogs and in the case of the log resolver specifically, we are using a the getTransactionReceipt call which does not provide a topic filter.

This PR attempts to pass the filter through the various places where logs are getting attached to the result set.

Edit: I've reopened this PR due to some misconfigurations in my local repository. This PR should be clean.

CLAassistant commented 5 years ago

CLA assistant check
All committers have signed the CLA.

kshinn commented 5 years ago

Ported fixes in #128