michaeljmarshall opened a new issue, #20023: URL: https://github.com/apache/pulsar/issues/20023
### Search before asking - [X] I searched in the [issues](https://github.com/apache/pulsar/issues) and found nothing similar. ### Motivation While working on https://github.com/apache/pulsar/pull/19989, I ran into test failures in cases that didn't seem to make sense. After additional research, I discovered this code: https://github.com/apache/pulsar/blob/3b76dd435d1bb7f5f933dbc15307255c7c928ca4/pulsar-broker/src/main/java/org/apache/pulsar/broker/web/PulsarWebResource.java#L508-L519 https://github.com/apache/pulsar/blob/3b76dd435d1bb7f5f933dbc15307255c7c928ca4/pulsar-broker/src/main/java/org/apache/pulsar/broker/web/PulsarWebResource.java#L536-L547 That code indicates that the logic for `getClusterDataIfDifferentCluster` changes depending on whether authorization is enabled. This is a surprising consequence, and it seems like we implicitly rely on the assumption that geo-replication will only be enabled when authorization is enabled. ### Solution See if it is possible to decouple authorization and `getClusterDataIfDifferentCluster`. ### Alternatives If we need to retain this dependency, we should document why. ### Anything else? _No response_ ### Are you willing to submit a PR? - [ ] 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]
