Skip to content

Remove the legacy REST API #7517

Closed
@amaury1093

Description

Summary

Remove the legacy REST API for v0.41.

Problem Definition

v0.40 introduced gRPC queries, which should be the preferred way of interacting with the app. For clients still wanting to use REST, all gRPC service methods are annotated to be accessible by REST + HTTP2, using grpc-gateway. Finally, for clients who don't support HTTP2 (e.g. browsers), a grpc-web-proxy is in the works.

All this removes the need of having a separate legacy REST API.

Proposal

  • in each module, remove the x/{module}/client/rest folder #9594.
  • remove all glue code between simapp/modules and the REST server #9594.

TODO:

  • in each module remove the legacy handler and querier #comment.

A clear migration path should also be provided.


For Admin Use

  • Not duplicate issue
  • Appropriate labels applied
  • Appropriate contributors tagged
  • Contributor assigned/self-assigned

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Assignees

Labels

Type: Code HygieneGeneral cleanup and restructuring of code to provide clarity, flexibility, and modularity.

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions