We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
For https://safe-transaction-base.safe.global/api/v1/safes/0x32e52896663De88a65c2D94917b006404415A89f/balances/ is really slow (until cache is warm).
After debugging it, we realised is not an RPC or database optimization issue.
The issue is that for a lot of tokens they don't exist in the database so we try to recover them from blockchain, with no luck as they don't hold usual ERC20 information like name or symbol: https://basescan.org/address/0x20b09dCb0Ca884448acE9DDB10D7851Bfe26C261#readContract
name
symbol
Method causing the bottleneck: https://github.com/safe-global/safe-transaction-service/blob/main/safe_transaction_service/history/services/balance_service.py#L293
The text was updated successfully, but these errors were encountered:
Add a table for not valid tokens
db0c5fa
- Not valid tokens are retrieved again and again after cache expires - Add a simple list of not valid tokens so RPC call is not required - Closes #2283
b68639d
f1021a7
Tested and verified after our latest deployment. Cheers!
Sorry, something went wrong.
Uxio0
Successfully merging a pull request may close this issue.
For https://safe-transaction-base.safe.global/api/v1/safes/0x32e52896663De88a65c2D94917b006404415A89f/balances/ is really slow (until cache is warm).
After debugging it, we realised is not an RPC or database optimization issue.
The issue is that for a lot of tokens they don't exist in the database so we try to recover them from blockchain, with no luck as they don't hold usual ERC20 information like
name
orsymbol
: https://basescan.org/address/0x20b09dCb0Ca884448acE9DDB10D7851Bfe26C261#readContractMethod causing the bottleneck: https://github.com/safe-global/safe-transaction-service/blob/main/safe_transaction_service/history/services/balance_service.py#L293
The text was updated successfully, but these errors were encountered: