cloudstateio / cloudstate

Distributed State Management for Serverless
https://cloudstate.io
Apache License 2.0
763 stars 97 forks source link

HTTP API default mappings #501

Closed pvlugter closed 3 years ago

pvlugter commented 3 years ago

The default mappings for the HTTP API are currently in the form /v1/MethodName, but method names can clash on multiple services. Noticed this when looking to include some tests for default mappings in the TCK proxy tests, but some of the different model services have the same method names.

Other solutions for HTTP/JSON transcoding, like Google Cloud Endpoints or grpc-gateway, use a default mapping of /fully.qualified.ServiceName/MethodName, such as /com.example.shoppingcart.ShoppingCart/GetCart, which is aligned with the fully qualified names used for gRPC calls. We should consider switching over to this for the default too.

sleipnir commented 3 years ago

I agre with proposal