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

No exception in this case in DubboHessian2SerializerImpl::deserializeRpcResult #7759

Closed jlspyaozhongkai closed 5 years ago

jlspyaozhongkai commented 5 years ago

https://github.com/envoyproxy/envoy/blob/3a7f9d9ae60abaa3546a5e28afba0815a2642782/source/extensions/filters/network/dubbo_proxy/dubbo_hessian2_serializer_impl.cc#L58

case RpcResponseType::ResponseWithException: case RpcResponseType::ResponseWithExceptionWithAttachments: case RpcResponseType::ResponseWithValue: result->setException(true); break;

and no value in this case

case RpcResponseType::ResponseNullValueWithAttachments:

stale[bot] commented 5 years ago

This issue has been automatically marked as stale because it has not had activity in the last 30 days. It will be closed in the next 7 days unless it is tagged "help wanted" or other activity occurs. Thank you for your contributions.

stale[bot] commented 5 years ago

This issue has been automatically closed because it has not had activity in the last 37 days. If this issue is still valid, please ping a maintainer and ask them to label it as "help wanted". Thank you for your contributions.