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

change liteserver storage to tonapi client to get jetton info in the … #302

Merged
merged 2 commits into from
Jan 19, 2024

Conversation

zakhar-petukhov
Copy link
Contributor

…rates method

@mr-tron mr-tron merged commit 7203605 into master Jan 19, 2024
1 check failed
@mr-tron mr-tron deleted the fix_rates branch January 19, 2024 12:40
@thedemons
Copy link
Contributor

Hey, could you explain the reasoning behind this change?

@mr-tron
Copy link
Collaborator

mr-tron commented Apr 7, 2024

good question

@thedemons
Copy link
Contributor

I'm getting a lot of errors because tonapi.io API is rate-limited, the whole point of running opentonapi is to use my own node & lite server, I don't understand why this change is necessary.

2024-04-08 13:00:36 {"time":"2024-04-08T06:00:36.334764054Z","level":"ERROR","prefix":"-","file":"market.go","line":"122","message":"[sendRequest] bad status code: 429, https://tonapi.io/v2/jettons/0:54bf44ac61bfc7a535fe5dd3a0149c1b99dda284c7115e75b52a99819850beb6"}
2024-04-08 13:00:36 {"time":"2024-04-08T06:00:36.334789996Z","level":"ERROR","prefix":"-","file":"market.go","line":"480","message":"[convertedDedustPoolResponse] failed to get jetton info: bad status code: 429"}

@bobbychen
Copy link

@thedemons Did you solved this problem?

@thedemons
Copy link
Contributor

@thedemons Did you solved this problem?

Yes, I just did it the easy way and change the url back to localhost, also keep in mind that when you run the docker using the provided command in READMD.md, it will use an image on docker hub instead of using your source. So you should build your own image instead.

@ImTheDeveloper
Copy link

Trying to build using my own file local just throws a bunch of new errors. These commits are brutal.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants