namada icon indicating copy to clipboard operation
namada copied to clipboard
verified publisher icon anoma

Implement event log and patch Namada to use it for tx querying

Open sug0 opened this issue 3 years ago • 5 comments

This PR patches in the diff from #587, adjusted to main.

The following changes are made:

  • The websockets client is removed from our code.
  • We Implement a log to store events emitted by ABCI+ calls to FinalizeBlock.
  • The log is hooked up to the shell, and is made queryable from two new ABCI query endpoints.
  • The client code now calls the new ABCI query endpoints to check the state (accepted/applied) of user txs.

Future work:

  • Tune default parameters of the event log.
  • Add a config option to change the number of kept events in the log.
  • Perhaps we could allow validators to disable the event log altogether.

sug0 avatar Oct 24 '22 13:10 sug0

pls update wasm

sug0 avatar Oct 24 '22 14:10 sug0

I've tested manual transfers a bit, locally, and the client seems to be working fine.

sug0 avatar Oct 25 '22 08:10 sug0

Putting this PR on hold until we fix e2e tests in https://github.com/anoma/namada/pull/723

sug0 avatar Nov 02 '22 16:11 sug0

pls update wasm

sug0 avatar Nov 03 '22 12:11 sug0

e2e tests are failing, but a fix is already available in #706

sug0 avatar Nov 03 '22 12:11 sug0