Skip to content
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

Patch/480 poolmanager errors and 481 remove fatal throws #482

Merged

Conversation

pharr117
Copy link
Collaborator

This PR adds the following:

  1. Removal of FTL errors when a parser fails -> We are preferring to keep the indexer running and to add to failed blocks table instead
  2. A fix for a broken set of poolmanager swaps -> If the token_swapped event is not found, try to parse the transfer event instead.

For (2), poolmanager manages a number of different swap types from a set of modules (Concentrated Liquidity, GAMM). It acts as a common wrapper around these. Osmosis added a new message type that uses WASM for swaps. This new update provides a different set of events.

See these 2 TX for some examples:

  1. Block 11509934: https://www.mintscan.io/osmosis/transactions/190EA05714BE7AF6CB6A95519D3CA81E7770A0B8A0E9091464CF1DA0F3E0006F
  2. Block 11509978: https://www.mintscan.io/osmosis/transactions/6F0DBD6061FED5D1A6027B6C41A3652797ACD5BDB893DC7D3E35CA22F2B3423D

Closes #480
Closes #481

…er instead of throwing FTL log. Update poolmanager swap parser to use transfer event in the case that the swap uses different events
@danbryan danbryan merged commit 1a4f47d into main Sep 19, 2023
3 checks passed
@danbryan danbryan deleted the patch/480-poolmanager-errors-and-481-remove-fatal-throws branch September 19, 2023 01:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants