You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pulsar.apache.org by mm...@apache.org on 2022/01/21 23:28:22 UTC

[pulsar] branch branch-2.8 updated: [Authorization] Return if namespace policies are read only (#12514)

This is an automated email from the ASF dual-hosted git repository.

mmarshall pushed a commit to branch branch-2.8
in repository https://gitbox.apache.org/repos/asf/pulsar.git


The following commit(s) were added to refs/heads/branch-2.8 by this push:
     new d565496  [Authorization] Return if namespace policies are read only (#12514)
d565496 is described below

commit d5654966a17cf378455a4782afb2b852d5f2383d
Author: Michael Marshall <mm...@apache.org>
AuthorDate: Fri Jan 21 17:08:23 2022 -0600

    [Authorization] Return if namespace policies are read only (#12514)
    
    * [Authorization Provider] Return early when namespace policies are read only
    
    * Remove typo fix to simplify cherry-picking
    
    (cherry picked from commit f1e72d6154332ca69e278005243ae6bde4e8c554)
---
 .../apache/pulsar/broker/authorization/PulsarAuthorizationProvider.java  | 1 +
 1 file changed, 1 insertion(+)

diff --git a/pulsar-broker-common/src/main/java/org/apache/pulsar/broker/authorization/PulsarAuthorizationProvider.java b/pulsar-broker-common/src/main/java/org/apache/pulsar/broker/authorization/PulsarAuthorizationProvider.java
index d276757..d0af3aa 100644
--- a/pulsar-broker-common/src/main/java/org/apache/pulsar/broker/authorization/PulsarAuthorizationProvider.java
+++ b/pulsar-broker-common/src/main/java/org/apache/pulsar/broker/authorization/PulsarAuthorizationProvider.java
@@ -294,6 +294,7 @@ public class PulsarAuthorizationProvider implements AuthorizationProvider {
             validatePoliciesReadOnlyAccess();
         } catch (Exception e) {
             result.completeExceptionally(e);
+            return result;
         }
 
         final String policiesPath = String.format("/%s/%s/%s", "admin", POLICIES, namespaceName.toString());