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

Zero balances in wallet #22203

Open
churik opened this issue Feb 27, 2025 · 3 comments
Open

Zero balances in wallet #22203

churik opened this issue Feb 27, 2025 · 3 comments
Assignees
Labels
wallet-core Issues for mobile wallet team
Milestone

Comments

@churik
Copy link
Member

churik commented Feb 27, 2025

Bug Report

Summary

For 10 minutes, all account balances displayed as 0.

Expected Behavior

Balances should be correctly displayed at all times.

Actual Behavior

  • Account balances were shown as 0 for 10 minutes.
  • Issue occurred twice in the past two days.

Screenshot

Image

Steps to Reproduce

  • No clear reproduction steps.
  • Occurred twice over the past two days.

Additional Information

Relevant Log Entry

2025-02-27T14:47:31.630Z	ERROR	market/market.go:106	Error fetching prices
{“error”: “cryptocompare.error: hystrix: timeout, coingecko.error: hystrix: timeout, cryptocompare-proxy.error: Get ***8: context deadline exceeded (Client.Timeout exceeded while awaiting headers)”}
@churik churik added the wallet-core Issues for mobile wallet team label Feb 27, 2025
@status-im status-im deleted a comment from Globalhub22 Feb 27, 2025
@status-im status-im deleted a comment Feb 27, 2025
@vkjr vkjr added this to the 2.33.0 milestone Feb 27, 2025
@friofry friofry assigned friofry and unassigned friofry Feb 27, 2025
@friofry
Copy link
Contributor

friofry commented Feb 27, 2025

It looks like we need to check our timeouts, in some cases it's 10, 20 seconds. It does not work perfectly because this timeout is shared between several providers that are called one after the other. I think it should be at least 30 sec * number of providers

@friofry
Copy link
Contributor

friofry commented Feb 28, 2025

2025-02-27T14:47:06.721Z ERROR    unknown         market/market.go:236 Error fetching prices
    error: 'cryptocompare.error: hystrix: timeout, coingecko.error: context deadline exceeded
      (Client.Timeout or context cancellation while reading body), cryptocompare-proxy.error:
      Get "https://prod.api.status.im/cryptocompare//data/all/coinlist": context deadline
      exceeded (Client.Timeout exceeded while awaiting headers)'

Image
	cb := circuitbreaker.NewCircuitBreaker(circuitbreaker.Config{
		Timeout:               10000,
		MaxConcurrentRequests: 100,
		SleepWindow:           300000,
		ErrorPercentThreshold: 25,
	})

So the http request will fail if the speed is less than 16Mbps

@friofry friofry self-assigned this Feb 28, 2025
@friofry
Copy link
Contributor

friofry commented Mar 2, 2025

Easy to reprodcue with 3G profile (780kbps) Network Link Conditioner from https://developer.apple.com/download/all/ (Additional Tools for Xcode)

Image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
wallet-core Issues for mobile wallet team
Projects
Status: No status
Development

No branches or pull requests

3 participants