dubbo_proxy parse hessian missing some case in switch

envoyproxy / envoy

Cloud-native high-performance edge/middle/service proxy

https://www.envoyproxy.io

Apache License 2.0

24.95k stars 4.8k forks source link

dubbo_proxy parse hessian missing some case in switch #13913

Closed serverglen closed 3 years ago

serverglen commented 3 years ago

HessianUtils::peekLong missing case 0xeb、0xec、0xed、0xfc、0xfd
HessianUtils::peekInt missing case 0xcc

zyfjeff commented 3 years ago

At present, the implementation of Hessian2 in the Envoy has been poorly completed. We plan to open the internal hessian2 code and then replace the implementation of Hessian2 in the Envoy.

serverglen commented 3 years ago

At present, the implementation of Hessian2 in the Envoy has been poorly completed. We plan to open the internal hessian2 code and then replace the implementation of Hessian2 in the Envoy.

@zyfjeff Thank for your reply, I communicated with you two weeks ago. you open the internal hessian2(refactor) code until the end of the year at least, I understand that this PR does not conflict with your refactoring code, because this is really a bug