[Bug] PIP-307 changes don't currently cover advertised listeners support (PIP-61/PIP-95)

[lhotari]

Search before asking

• I searched in the issues and found nothing similar.

Version

master

Minimal reproduce step

this code doesn't support advertised listeners:

pulsar/pulsar-broker/src/main/java/org/apache/pulsar/broker/service/ServerCnx.java

Lines 3149 to 3150 in 09559c5

	assignedBrokerLookupData.map(BrokerLookupData::pulsarServiceUrl).orElse(null),
	assignedBrokerLookupData.map(BrokerLookupData::pulsarServiceUrlTls).orElse(null)));

What did you expect to see?

PIP-307 should support advertised listeners feature

What did you see instead?

The default internal listener addresses are used.

Anything else?

There is PIP-338 in discussion which highlights some of the problems.

Are you willing to submit a PR?

• I'm willing to submit a PR!

[lhotari]

There's also a need for SNI proxy support to cover all connectivity options. I think that it is commonly used together with Pulsar's SNI proxy feature.