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

execute_read_only... endpoints has 2 usages #4677

Open
AurelienFT opened this issue Apr 11, 2024 · 0 comments
Open

execute_read_only... endpoints has 2 usages #4677

AurelienFT opened this issue Apr 11, 2024 · 0 comments
Labels
api Issues related to the API enhancement New feature or request

Comments

@AurelienFT
Copy link
Contributor

Context

execute_read_only_bytecode and execute_read_only_call are two endpoints that has 2 usages that collide :

  • Calling read_only functions to gather informations on a smart-contract
  • Simulate an operation gather the changes it make in the network and the gas used

Problem

This two use case causes problem of coherence issues. For example we had the issue with the minimal fees that was checked even in the usecase of just a read_only which is a non-sense, but we want to catch a too low fee in case of a simulation to be more close to a real operation.

@AurelienFT AurelienFT added enhancement New feature or request api Issues related to the API labels Apr 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api Issues related to the API enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant