pulsar-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From GitBox <...@apache.org>
Subject [GitHub] sijie opened a new pull request #2382: Issue 2283: Improve error message if authorization is not enabled
Date Wed, 15 Aug 2018 20:38:41 GMT
sijie opened a new pull request #2382: Issue 2283: Improve error message if authorization is
not enabled
URL: https://github.com/apache/incubator-pulsar/pull/2382
 
 
    ### Motivation
   
    Fixes #2283.
   
    If authorizationEnabled is not enabled, authorization service is not set. All the accesses
to this authorization service will throw NPE and clients will receive Internal Server Error.
The message is meaningless. It is better to return a more meaningful message.
   
    ### Changes
   
    Return `Not implemented` if authorization is not enabled.
   
   
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

Mime
View raw message