addisonj opened a new issue #10124:
URL: https://github.com/apache/pulsar/issues/10124


   **Describe the bug**
   In the code, `webServicePort` is marked as optional and in many places in 
the code, we properly check if it is present before starting a non TLS listener.
   
   However, other paths in code are not properly checking this. For example:
   
https://github.com/apache/pulsar/blob/bf00673fa0a2b35b22c00e47a72fca1d28a6f05f/pulsar-broker/src/main/java/org/apache/pulsar/broker/namespace/NamespaceService.java#L615-L618
   This not the only instance, there are a number of other places, such as when 
functions is enabled.
   
   **To Reproduce**
   Steps to reproduce the behavior:
   1. Run a broker with only TLS (set `webServicePort=` to empty)
   
   Perform some actions and potentially get a crash
   
   **Expected behavior**
   We should not required `webServicePort` to be required and we should be 
properly checking everything for the optional usage.
   
   
   


-- 
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.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to