[GitHub] [pulsar] addisonj opened a new issue #10124: Don't require webServicePort to allow for TLS only listeners

[GitBox <gi...@apache.org>]

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:
users@infra.apache.org

[GitHub] [pulsar] codelipenghui commented on issue #10124: Don't require webServicePort to allow for TLS only listeners

[GitBox <gi...@apache.org>]

codelipenghui commented on issue #10124:
URL: https://github.com/apache/pulsar/issues/10124#issuecomment-1058891745


   The issue had no activity for 30 days, mark with Stale label.


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