-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
gRPC Transaction Simulation Service #5922
Comments
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
I started working on a protobuf service definition for this. Initially, I started by just copying the current transaction generation In discussing with my team, I am rethinking this altogether. My assessment is that REST transaction generation is basically an anti-pattern. Clients now have the full protobuf Signing transaction content returned from a possibly untrusted service should be avoided at all costs. Maybe the current REST tx endpoints were needed given the situation with amino. But all of the users I talked with when designing the current I propose instead that we create a simple gRPC service for How does this sound as an alternative? |
ACK from me @aaronc -- less work as well 👍 |
Summary
As discussed in #5894 (comment), we need an approach for REST/gRPC-based transaction generation to either upgrade or replace the existing REST tx routes. The approach adopted here is to provide a gRPC tx simulation service (see #5922 (comment))
cosmos.tx.Service
gRPC serviceThe text was updated successfully, but these errors were encountered: