alpreu opened a new issue, #21012: URL: https://github.com/apache/pulsar/issues/21012
### Search before asking - [X] I searched in the [issues](https://github.com/apache/pulsar/issues) and found nothing similar. ### Version Pulsar 3.1.0 but also broken on master ### Minimal reproduce step 1. Start pulsar standalone with TLS configured using e.g. port 8081 for TLS 2. Run pulsarAdmin.brokers().getActiveBrokers() ### What did you expect to see? The return value should be `localhost:8081` (using TLS port) ### What did you see instead? The return value is `localhost:8080` (not using TLS port) ### Anything else? The value is ultimately retrieved from `PulsarService.getLookupServiceAddress`. I believe the code below should be the other way around, first checking if a TLS address is available and if not fall back to the non tls one: https://github.com/apache/pulsar/blob/master/pulsar-broker/src/main/java/org/apache/pulsar/broker/PulsarService.java#L1740-L1743 ### Are you willing to submit a PR? - [X] I'm willing to submit a PR! -- 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]
