cointop-sh / cointop

A fast and lightweight interactive terminal based UI application for tracking cryptocurrencies 🚀
https://cointop.sh
Apache License 2.0
3.98k stars 311 forks source link

no enough data for chart #226

Closed somera closed 2 years ago

somera commented 2 years ago

I checked the 1.6.9 version and I get

image

Same with self compiled sources and the docker v1.6.9 and v1.6.8 version.

lyricnz commented 2 years ago

I was getting the same thing last night. I guess coingecko (default backend) may have been having some issues. I'll add some code for doing (more) verbose network logging.

lyricnz commented 2 years ago

It looks like Cloudflare (CDN) are blocking requests to the endpoint we're using to fetch the data for the main chart

https://www.coingecko.com/market_cap/total_charts_data?duration=365&locale=en&vs_currency=usd

This is technically not part of the API (it's part of the website), and I don't know how we can get the same data from the API. The fact that we hit the same endpoint 10+ times in fairly quick succession probably means Cloudflare thinks it's a DOS.

lyricnz commented 2 years ago

Seems to be working OK today for me. You?

somera commented 2 years ago

Looks good. Thx.