You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@pulsar.apache.org by GitBox <gi...@apache.org> on 2021/09/16 21:28:14 UTC

[GitHub] [pulsar] dinghram commented on pull request #12057: Fixed ProxyConnection to check for existence of auth_data field

dinghram commented on pull request #12057:
URL: https://github.com/apache/pulsar/pull/12057#issuecomment-921264489


   This change fixed the original problem, but then caused a npe in OneStageAuthenticationState.java:
   new String(authData.getBytes(), UTF_8)
   
   I propose that we change line 318 to:
   AuthData clientData = AuthData.of(connect.hasAuthData() ? connect.getAuthData() : "".getBytes());


-- 
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: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org