search

Concordium / concordium-reference-wallet-android

Reference Android wallet for the Concordium blockchain
Apache License 2.0
12 stars 2 forks source link

Missing shielded transfers #62

Closed orhoj closed 1 year ago

orhoj commented 2 years ago

Bug Description In some cases the mobile wallet does not display shielded transfers.

Steps to Reproduce

  1. Create a fresh account.
  2. Send a shielded transfer to the account.
  3. Send 200 simple transfers to the account.
  4. Go into the "Shielded balance" and see that there are no shielded transfers shown, even though we sent one in step 2.

Expected Result The shielded transfer from step 2) should be visible.

Actual Result The shielded transfer list is empty. From the log files on the wallet proxy we can see that it stops querying for transactions before it reaches the query that would contain the shielded transfer.

It queries:

  1. GET /v1/accTransactions/3R14AK9HisDLnFkph2vUq2DFekivkhtPZrTTfi6feLQNKy17VH?order=desc&limit=100&includeRewards=all
  2. GET /v1/accTransactions/3R14AK9HisDLnFkph2vUq2DFekivkhtPZrTTfi6feLQNKy17VH?order=desc&from=20635894&limit=100&includeRewards=all

The final transaction in the 2nd query has id = 20635894, and querying with from=20635894 the missing shielded transaction is found, but the mobile wallet does not perform this query. So it appears that the escape clause for stopping to pull transactions from the wallet proxy is a little off.

Versions

orhoj commented 2 years ago

I do not know if a similar issue exists for the iOS mobile wallet.

[EDIT] The issue does not appear to be present on iOS.

concordium-cl commented 1 year ago

Moved to https://concordium.atlassian.net/browse/CBW-717