envoyproxy/envoy
Ver no GitHubgrpc async client should catch proto cast exception
Open
#5.409 aberto em 24 de dez. de 2018
enhancementhelp wanted
Métricas do repositório
- Stars
- (27.997 stars)
- Métricas de merge de PR
- (Mesclagem média 8d) (378 fundiu PRs em 30d)
Description
Title: grpc async client should catch proto cast exception
Description:
grpc async client proto cast catches no exception as follows:
include\envoy\grpc\async_client.h
void onReceiveMessageUntyped(ProtobufTypes::MessagePtr&& message) override {
onReceiveMessage(std::unique_ptr<ResponseType>(dynamic_cast<ResponseType*>(message.release())));
}
which may crashs the process when the received message is not standard, like 00 00 00 00 00.
And should be modified as follows:
void onReceiveMessageUntyped(ProtobufTypes::MessagePtr&& message) override {
try {
onReceiveMessage(std::unique_ptr<ResponseType>(dynamic_cast<ResponseType*>(message.release())));
} catch(const std::exception& e) {
std::cout << "wrong message received, error: " << e.what() << std::endl;
}
}