-
Couldn't load subscription status.
- Fork 5
refactor: use standard rpc style #9
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
Conversation
# Conflicts: # docs/swagger/campusapis/health/v1/health.swagger.json # docs/swagger/consultingapis/faq/v1/faq.swagger.json
打算什么时候合并
需要所有相关依赖服务都迁移至新的proto定义且完成测试
该pr需要code review
# Conflicts: # campusapis/staff/v1/campus.pb.go # campusapis/staff/v1/campus.pb.gw.go # campusapis/staff/v1/campus.proto # campusapis/staff/v1/campus_grpc.pb.go # docs/swagger/campusapis/staff/v1/campus.swagger.json
# Conflicts: # aggregatingapis/infostream/v1/infostream.pb.go
...to dev-standard-rpc
@fdkevin0 给前端的数据多包了一层怎么处理,是在gateway之类的地方把外壳脱了还是直接让前端改
@fdkevin0 给前端的数据多包了一层怎么处理,是在gateway之类的地方把外壳脱了还是直接让前端改
什么叫多包了一层,给一个样例
之前我是在salmon_base中grpc_gateway中劫持http返回体,然后强行包一层的,这会导致rpc定义与实际不符
在新分支内我已经改用proto定义返回体最外层,然后使用grpc中间件劫持error并注入grpc返回体最外层
什么叫多包了一层,给一个样例
之前我是在salmon_base中grpc_gateway中劫持http返回体,然后强行包一层的,这会导致rpc定义与实际不符
在新分支内我已经改用proto定义返回体最外层,然后使用grpc中间件劫持error并注入grpc返回体最外层
哦哦懂了,之前理解错了
# Conflicts: # campusapis/staff/v1/campus.pb.go # docs/swagger/campusapis/staff/v1/campus.swagger.json
有亿点review不动😥
Uh oh!
There was an error while loading. Please reload this page.