nodece commented on code in PR #122:
URL: https://github.com/apache/pulsar-client-cpp/pull/122#discussion_r1031147715
##########
lib/ClientConnection.cc:
##########
@@ -1709,6 +1710,9 @@ void ClientConnection::checkServerError(ServerError
error) {
// https://github.com/apache/pulsar/pull/274
closeSocket();
break;
+ case proto::ServerError::AuthorizationError:
Review Comment:
When receiving the AuthenticationError or AuthorizationError, I think we
should disconnect the connection.
Right now, we only handle the AuthenticationError in Java client. See
https://github.com/apache/pulsar/blob/94a6d3642800ac48b779168cbe17bb505b7c1f96/pulsar-client/src/main/java/org/apache/pulsar/client/impl/ClientCnx.java#L757
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
