[General][Java SDK (Component)] Triple protocol content-type specification

[chickenlj]

Pre-check

• I am sure that all the content I provide is in English.

Apache Dubbo Component

Java SDK (apache/dubbo)

Details

Code of Conduct

• I agree to follow this project's Code of Conduct

[chickenlj]

刘军:[Image]
刘军:java这边，现在使用triple wrapper模式，client 是用会发送上面这个content-type吗？

貌似是不用的，go那边是用的（如上图所示），导致调用不通
刘军:java client 是会用 grpc+hessian2 吗？
刘军:现在的结论是：

1. go-client 调用 java-server 是通的
2. java-client 调用 go-server 是不通的，因为 go-server 那边识别到的content-type是 grpc+proto，导致handler定位不对
刘军:至于第1个场景为何会通，还没看content-type是啥，大概率 go-client 发送的是 grpc+hessian2，如果是那样的话，说明java-server是能识别这个的content-type的
陈有为-earthchen:JAVA侧应该目前只有app/grpc
刘军:@陈有为-earthchen  咱们之前计划的目标是啥来着？是不是加上grpc/hessian2合适
刘军:[Image]
陈有为-earthchen:这种当时说把 grpc 去掉
陈有为-earthchen:tri+xx
刘军:以上，java server收到go client 发过来的grpc+hessian2是能通的，感觉java这边实现不太严谨
刘军:@陈有为-earthchen  可以呀，那就按照这个，咱们捋一波。我回头约一下go那边一起，咱们把这点从头到尾捋一遍[NosePick]
陈有为-earthchen:老版的wrapper还是grpc
刘军:再就是上面的大小写问题，看起来java server这边还好的，go server那边又点问题：
1. 路径上go client发送的是 /org.apache.dubbo.tri.hessian2.api.GreetingsService/Greet，java server是能定位到 greet 方法的
2. 路径上java client发送的是 /org.apache.dubbo.tri.hessian2.api.GreetingsService/greet，go server那边定位不到 Greet 方法
刘军:就这俩问题，其他的看起来都没啥问题了：
1. content-type 
2. 路径大小写匹配
陈有为-earthchen:我记得这跟pb文件怎么写的有关系
陈有为-earthchen:而且分你是用stub还是别的
陈有为-earthchen:非stub本来就应该是啥就生成啥
陈有为-earthchen:但是pb的规范是方法名大写
刘军:这个大小写问题其实是从java interface模式引起的，因为java手写接口定义的一般都小写greet，go那边手写/工具生成都得是大写Greet（标记public），天然大小写不一致
陈有为-earthchen:这种不应该做处理
刘军:这个我看先把go那边改成路径匹配大小写不敏感
刘军:@陈有为-earthchen  是的，java就不管，直接发过去。go server那边我看看改改，遇到小写path给匹配到大写方法上～
陈有为-earthchen:想处理倒是很简单
陈有为-earthchen:可是应该不应该处理是个问题
陈有为-earthchen:理论上我们header大小写不敏感就好了
刘军:没啥好办法了，为了互通go这边只能处理下，java过来的都是小写咋办
刘军:我在go这边只处理方法大小写不敏感（能识别到位置），遇到小写的就转发到大写public方法，其他的继续保持不动
刘军:@陈有为-earthchen  java这边，没用content-type，server这边收到请求怎么识别的是pb还是wrapper来着？
陈有为-earthchen:根据metadata来判断
刘军:[Image]
java client发过来确实是 grpc+proto，后面又某个位置区分了？
刘军:看到了，看起来是message里解码特定位置判断的
陈有为-earthchen:不是的啊  是根据方法名拿到md
陈有为-earthchen:md里就说了
刘军:[Image]
找到前置位置了
刘军:当前java server对收到的content-type没起过滤作用？比如收到appliatin/grpc+hessian2应该是拒绝的，因为当前实现不支持
陈有为-earthchen:现在似乎只判断了grpc吧
刘军:[Image]
陈有为-earthchen:看下 support 方法
陈有为-earthchen:判断 startwith 了
刘军:[Image]
刘军:大概搞清楚了，等我找go那边的人先看看，短期让那边也先做一个 isProtobuf判断兼容下。长期等看咱们怎么面向终态统一一下，就是感觉兼容老版本头疼可能
陈有为-earthchen:java 存在的久  肯定得 go 去兼容
陈有为-earthchen:长期我建议定义 tri+xxx
陈有为-earthchen:把 wrapper 干掉
陈有为-earthchen:或者直接参考 app/json
陈有为-earthchen:body 直接放序列化后的 byte