| commit | 3c82d27193e1c687ec5f14b1a367ae6441d390dd | [log] [tgz] |
|---|---|---|
| author | Yunze Xu <xyzinfernity@163.com> | Sun Jul 23 11:59:19 2023 +0800 |
| committer | GitHub <noreply@github.com> | Sun Jul 23 11:59:19 2023 +0800 |
| tree | 87eb45eb05626875605b88bf18dde0611ca4f9e6 | |
| parent | e7ae9e2544cced28eb4c2ae6c4b2f01a5879a318 [diff] |
[fix][broker] Fix the built-in admin failed to delete a topic with a custom authz provider (#20848)
diff --git a/pulsar-broker-common/src/main/java/org/apache/pulsar/broker/authorization/AuthorizationProvider.java b/pulsar-broker-common/src/main/java/org/apache/pulsar/broker/authorization/AuthorizationProvider.java index 9ea49fe..24b94ef 100644 --- a/pulsar-broker-common/src/main/java/org/apache/pulsar/broker/authorization/AuthorizationProvider.java +++ b/pulsar-broker-common/src/main/java/org/apache/pulsar/broker/authorization/AuthorizationProvider.java
@@ -436,9 +436,7 @@ * @return CompletableFuture<Void> */ default CompletableFuture<Void> removePermissionsAsync(TopicName topicName) { - return FutureUtil.failedFuture(new IllegalStateException( - String.format("removePermissionsAsync on topicName %s is not supported by the Authorization", - topicName))); + return CompletableFuture.completedFuture(null); } /**
diff --git a/pulsar-broker/src/test/java/org/apache/pulsar/broker/auth/AuthorizationWithAuthDataTest.java b/pulsar-broker/src/test/java/org/apache/pulsar/broker/auth/AuthorizationWithAuthDataTest.java index b69ccc4..bb0461d 100644 --- a/pulsar-broker/src/test/java/org/apache/pulsar/broker/auth/AuthorizationWithAuthDataTest.java +++ b/pulsar-broker/src/test/java/org/apache/pulsar/broker/auth/AuthorizationWithAuthDataTest.java
@@ -270,6 +270,7 @@ admin.topics().createNonPartitionedTopic(nonPartitionedTopic); admin.lookups().lookupPartitionedTopic(partitionedTopic); admin.lookups().lookupTopic(nonPartitionedTopic); + admin.topics().delete(nonPartitionedTopic); } @Test