Skip to content
This repository has been archived by the owner on Feb 2, 2024. It is now read-only.

Difference in Tx details in Prod and in Barn #142

Closed
elena-zh opened this issue Jun 23, 2022 · 3 comments
Closed

Difference in Tx details in Prod and in Barn #142

elena-zh opened this issue Jun 23, 2022 · 3 comments
Labels
app:Explorer Explorer App Bug Something isn't working Low Severity indicator for defects. It won't cause any major break-down of the system

Comments

@elena-zh
Copy link

Tx hash:
0xc247e6ecdad9a20135400844b123f20b18e6fe2e69001d5b9bc919fa2fb0ead4

Prod shows 4 transactions:
image.png

DEV shows 5 transactions
image.png

BV diagram shows 4 nodes for both diagrams
image.png

Expected: number of transactions should be consistent?

@elena-zh elena-zh added app:Explorer Explorer App Bug Something isn't working Low Severity indicator for defects. It won't cause any major break-down of the system labels Jun 23, 2022
@ramirotw
Copy link
Contributor

Tx hash: 0xc247e6ecdad9a20135400844b123f20b18e6fe2e69001d5b9bc919fa2fb0ead4

Prod shows 4 transactions: image.png

DEV shows 5 transactions image.png

This is ok:

The production site explorer.cow.fi is not merging responses yet, so it will only display the 4 orders from the prod env. On the other hand, the Release v2.9.0 is merging responses so it's fine that it shows 5.

BUT, what is actually not ok is the onchain data, which doesn't have the STRONG -> ETH order.

So this is weird:

  1. https://barn.api.cow.fi/mainnet/api/v1/transactions/0xc247e6ecdad9a20135400844b123f20b18e6fe2e69001d5b9bc919fa2fb0ead4/orders returns order id #0x4880647 which is incorrect because that order wasn't included in that tx

https://barn.api.cow.fi/mainnet/api/v1/trades/?orderUid=0x4880647d08051b755ef694c509df1a701047fc734736171fe2378d1fad20d68e6729a00e0cdf0562fa1fb246bc56bbeb6dbbc06262a4796a has the correct tx hash which is 0x4e50b8d115d449b14c92ef57f66db76269d4ab4197c610d4e1810e8cd6eff7bf

It seems that the Barn API added the order #0x4880647 incorrectly to the batch https://etherscan.io/tx/0xc247e6ecdad9a20135400844b123f20b18e6fe2e69001d5b9bc919fa2fb0ead4

@anxolin @alfetopito @nlordell

@nlordell
Copy link
Contributor

This looks like this known issue: cowprotocol/services#89

@elena-zh
Copy link
Author

Prod and Dev show the same data now for the transaction in the description.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
app:Explorer Explorer App Bug Something isn't working Low Severity indicator for defects. It won't cause any major break-down of the system
Projects
None yet
Development

No branches or pull requests

3 participants