You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pulsar.apache.org by GitBox <gi...@apache.org> on 2020/04/03 13:10:53 UTC

[GitHub] [pulsar] KannarFr opened a new pull request #6661: [Issue 6658] support multiple authz providers

KannarFr opened a new pull request #6661: [Issue 6658] support multiple authz providers
URL: https://github.com/apache/pulsar/pull/6661
 
 
   Fixes https://github.com/apache/pulsar/issues/6658
   
   ### Motivation
   
   We are developping a custom authz provider and we don't want breaking changes on current authz usages.
   
   ### Modifications
   
   I reproduced the way it's done for multiple authentication providers.
   
   ### Verifying this change
   
   - [ ] Make sure that the change passes the CI checks.
   
   This change is already covered by existing tests, such as *(please describe tests)*.
   
   ### Does this pull request potentially affect one of the following parts:
   
   ### Documentation
   
     - Does this pull request introduce a new feature? yes
     - If yes, how is the feature documented? docs JavaDocs
   

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [pulsar] KannarFr commented on a change in pull request #6661: [Issue 6658] support multiple authz providers

Posted by GitBox <gi...@apache.org>.
KannarFr commented on a change in pull request #6661: [Issue 6658] support multiple authz providers
URL: https://github.com/apache/pulsar/pull/6661#discussion_r403856174
 
 

 ##########
 File path: pulsar-broker-common/src/main/java/org/apache/pulsar/broker/authorization/AuthorizationService.java
 ##########
 @@ -70,17 +90,38 @@ public AuthorizationService(ServiceConfiguration conf, ConfigurationCacheService
         }
     }
 
+    private static <T> CompletableFuture<T> anyMatch(List<? extends CompletionStage<? extends T>> l,
 
 Review comment:
   ```
   /**
     * @return the authorization methods names supported by this provider
     */
   default String getAuthZClassname() {
       return "org.apache.pulsar.broker.authorization.PulsarAuthorizationProvider";
   }
   ```

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [pulsar] KannarFr commented on a change in pull request #6661: [Issue 6658] support multiple authz providers

Posted by GitBox <gi...@apache.org>.
KannarFr commented on a change in pull request #6661: [Issue 6658] support multiple authz providers
URL: https://github.com/apache/pulsar/pull/6661#discussion_r403853332
 
 

 ##########
 File path: pulsar-broker-common/src/main/java/org/apache/pulsar/broker/authorization/AuthorizationService.java
 ##########
 @@ -70,17 +90,38 @@ public AuthorizationService(ServiceConfiguration conf, ConfigurationCacheService
         }
     }
 
+    private static <T> CompletableFuture<T> anyMatch(List<? extends CompletionStage<? extends T>> l,
 
 Review comment:
   @sijie considering https://github.com/apache/pulsar/pull/6661#issuecomment-608543559, maybe we should just define the authz provider to use on the auth provider, so dynamically use the right authz provider depending on the auth provider.

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [pulsar] KannarFr commented on a change in pull request #6661: [Issue 6658] support multiple authz providers

Posted by GitBox <gi...@apache.org>.
KannarFr commented on a change in pull request #6661: [Issue 6658] support multiple authz providers
URL: https://github.com/apache/pulsar/pull/6661#discussion_r403855179
 
 

 ##########
 File path: pulsar-broker-common/src/main/java/org/apache/pulsar/broker/authorization/AuthorizationService.java
 ##########
 @@ -70,17 +90,38 @@ public AuthorizationService(ServiceConfiguration conf, ConfigurationCacheService
         }
     }
 
+    private static <T> CompletableFuture<T> anyMatch(List<? extends CompletionStage<? extends T>> l,
 
 Review comment:
   So put `Set<String> getAuthZMethodsNames();` on `AuthenticationProvider` and use the defined authZ provider depending on the auth provider in `AuthorizationService`. WDYT? @sijie @merlimat 

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [pulsar] sijie commented on a change in pull request #6661: [Issue 6658] support multiple authz providers

Posted by GitBox <gi...@apache.org>.
sijie commented on a change in pull request #6661: [Issue 6658] support multiple authz providers
URL: https://github.com/apache/pulsar/pull/6661#discussion_r403747087
 
 

 ##########
 File path: pulsar-broker-common/src/main/java/org/apache/pulsar/broker/authorization/AuthorizationService.java
 ##########
 @@ -70,17 +90,38 @@ public AuthorizationService(ServiceConfiguration conf, ConfigurationCacheService
         }
     }
 
+    private static <T> CompletableFuture<T> anyMatch(List<? extends CompletionStage<? extends T>> l,
 
 Review comment:
   `anyMatch` doesn't seem to be right to me when multiple authentication providers are applied to. Is it better to check them in sequence? 

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [pulsar] KannarFr commented on a change in pull request #6661: [Issue 6658] support multiple authz providers

Posted by GitBox <gi...@apache.org>.
KannarFr commented on a change in pull request #6661: [Issue 6658] support multiple authz providers
URL: https://github.com/apache/pulsar/pull/6661#discussion_r403873632
 
 

 ##########
 File path: pulsar-broker-common/src/main/java/org/apache/pulsar/broker/authorization/AuthorizationService.java
 ##########
 @@ -70,17 +90,38 @@ public AuthorizationService(ServiceConfiguration conf, ConfigurationCacheService
         }
     }
 
+    private static <T> CompletableFuture<T> anyMatch(List<? extends CompletionStage<? extends T>> l,
 
 Review comment:
   Or just a simple way: as it's done for auth providers, just verify there is at least one authZ provider that authorizes the request this is why the `anyMatch`.

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [pulsar] sijie commented on issue #6661: [Issue 6658] support multiple authz providers

Posted by GitBox <gi...@apache.org>.
sijie commented on issue #6661: [Issue 6658] support multiple authz providers
URL: https://github.com/apache/pulsar/pull/6661#issuecomment-609994768
 
 
   @KannarFr cool

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [pulsar] KannarFr commented on a change in pull request #6661: [Issue 6658] support multiple authz providers

Posted by GitBox <gi...@apache.org>.
KannarFr commented on a change in pull request #6661: [Issue 6658] support multiple authz providers
URL: https://github.com/apache/pulsar/pull/6661#discussion_r403856174
 
 

 ##########
 File path: pulsar-broker-common/src/main/java/org/apache/pulsar/broker/authorization/AuthorizationService.java
 ##########
 @@ -70,17 +90,38 @@ public AuthorizationService(ServiceConfiguration conf, ConfigurationCacheService
         }
     }
 
+    private static <T> CompletableFuture<T> anyMatch(List<? extends CompletionStage<? extends T>> l,
 
 Review comment:
   ```
   /**
     * @return the authorization classname supported by this provider
     */
   default String getAuthZClassname() {
       return "org.apache.pulsar.broker.authorization.PulsarAuthorizationProvider";
   }
   ```

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [pulsar] KannarFr commented on issue #6661: [Issue 6658] support multiple authz providers

Posted by GitBox <gi...@apache.org>.
KannarFr commented on issue #6661: [Issue 6658] support multiple authz providers
URL: https://github.com/apache/pulsar/pull/6661#issuecomment-608543559
 
 
   The best way should be to specify authZ provider to use with authprovider. As multiple auth providers are supported, multiple authZ providers should be defined consequently.
   
   The simplest way here, is if any authZ provider authorizes then go. 
   
   WDYT?

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [pulsar] KannarFr commented on issue #6661: [Issue 6658] support multiple authz providers

Posted by GitBox <gi...@apache.org>.
KannarFr commented on issue #6661: [Issue 6658] support multiple authz providers
URL: https://github.com/apache/pulsar/pull/6661#issuecomment-609981611
 
 
   Close for now, the best solution would be to write an AuthorizationProvider that take a list of AuthorizationProviders. No code to change, just add a master authz provider.

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [pulsar] KannarFr commented on a change in pull request #6661: [Issue 6658] support multiple authz providers

Posted by GitBox <gi...@apache.org>.
KannarFr commented on a change in pull request #6661: [Issue 6658] support multiple authz providers
URL: https://github.com/apache/pulsar/pull/6661#discussion_r403855179
 
 

 ##########
 File path: pulsar-broker-common/src/main/java/org/apache/pulsar/broker/authorization/AuthorizationService.java
 ##########
 @@ -70,17 +90,38 @@ public AuthorizationService(ServiceConfiguration conf, ConfigurationCacheService
         }
     }
 
+    private static <T> CompletableFuture<T> anyMatch(List<? extends CompletionStage<? extends T>> l,
 
 Review comment:
   So put `Set<String> getAuthZMethodsNames();` on `AuthenticationProvider` and use the defined authZ providers depending on the auth provider in `AuthorizationService`. WDYT? @sijie @merlimat 

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [pulsar] KannarFr commented on a change in pull request #6661: [Issue 6658] support multiple authz providers

Posted by GitBox <gi...@apache.org>.
KannarFr commented on a change in pull request #6661: [Issue 6658] support multiple authz providers
URL: https://github.com/apache/pulsar/pull/6661#discussion_r403856174
 
 

 ##########
 File path: pulsar-broker-common/src/main/java/org/apache/pulsar/broker/authorization/AuthorizationService.java
 ##########
 @@ -70,17 +90,38 @@ public AuthorizationService(ServiceConfiguration conf, ConfigurationCacheService
         }
     }
 
+    private static <T> CompletableFuture<T> anyMatch(List<? extends CompletionStage<? extends T>> l,
 
 Review comment:
   ```
   /**
     * @return the authorization methods names supported by this provider
     */
   default Set<String> getAuthZClassesNames() {
       return Sets.newHashSet("org.apache.pulsar.broker.authorization.PulsarAuthorizationProvider");
   }
   ```

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [pulsar] KannarFr commented on a change in pull request #6661: [Issue 6658] support multiple authz providers

Posted by GitBox <gi...@apache.org>.
KannarFr commented on a change in pull request #6661: [Issue 6658] support multiple authz providers
URL: https://github.com/apache/pulsar/pull/6661#discussion_r403855179
 
 

 ##########
 File path: pulsar-broker-common/src/main/java/org/apache/pulsar/broker/authorization/AuthorizationService.java
 ##########
 @@ -70,17 +90,38 @@ public AuthorizationService(ServiceConfiguration conf, ConfigurationCacheService
         }
     }
 
+    private static <T> CompletableFuture<T> anyMatch(List<? extends CompletionStage<? extends T>> l,
 
 Review comment:
   So put `Set<String> getAuthZClassesNames();` on `AuthenticationProvider` and use the defined authZ providers depending on the auth provider in `AuthorizationService`. WDYT? @sijie @merlimat 

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [pulsar] merlimat commented on issue #6661: [Issue 6658] support multiple authz providers

Posted by GitBox <gi...@apache.org>.
merlimat commented on issue #6661: [Issue 6658] support multiple authz providers
URL: https://github.com/apache/pulsar/pull/6661#issuecomment-608524952
 
 
   What is the semantic of multiple AuthZ providers? This is probably different in comparison with Authentication (eg: you can pass the credentials in multiple forms). 
   
   For example, what would mean that providerA says I don't have access and providerB says I do?

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [pulsar] KannarFr edited a comment on issue #6661: [Issue 6658] support multiple authz providers

Posted by GitBox <gi...@apache.org>.
KannarFr edited a comment on issue #6661: [Issue 6658] support multiple authz providers
URL: https://github.com/apache/pulsar/pull/6661#issuecomment-610002085
 
 
   (Due to a talk with @sijie ;))

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [pulsar] KannarFr commented on a change in pull request #6661: [Issue 6658] support multiple authz providers

Posted by GitBox <gi...@apache.org>.
KannarFr commented on a change in pull request #6661: [Issue 6658] support multiple authz providers
URL: https://github.com/apache/pulsar/pull/6661#discussion_r403856174
 
 

 ##########
 File path: pulsar-broker-common/src/main/java/org/apache/pulsar/broker/authorization/AuthorizationService.java
 ##########
 @@ -70,17 +90,38 @@ public AuthorizationService(ServiceConfiguration conf, ConfigurationCacheService
         }
     }
 
+    private static <T> CompletableFuture<T> anyMatch(List<? extends CompletionStage<? extends T>> l,
 
 Review comment:
   ```
   /**
     * @return the authorization methods names supported by this provider
     */
   default Set<String> getAuthZMethodsNames() {
       return Sets.newHashSet("org.apache.pulsar.broker.authorization.PulsarAuthorizationProvider");
   }
   ```

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [pulsar] codelipenghui commented on a change in pull request #6661: [Issue 6658] support multiple authz providers

Posted by GitBox <gi...@apache.org>.
codelipenghui commented on a change in pull request #6661: [Issue 6658] support multiple authz providers
URL: https://github.com/apache/pulsar/pull/6661#discussion_r403016565
 
 

 ##########
 File path: pulsar-broker-common/src/main/java/org/apache/pulsar/broker/authorization/AuthorizationService.java
 ##########
 @@ -70,17 +90,53 @@ public AuthorizationService(ServiceConfiguration conf, ConfigurationCacheService
         }
     }
 
+    public static <T> CompletableFuture<List<T>> allAsList(List<? extends CompletionStage<? extends T>> stages) {
 
 Review comment:
   Is this method still be used?

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [pulsar] KannarFr commented on issue #6661: [Issue 6658] support multiple authz providers

Posted by GitBox <gi...@apache.org>.
KannarFr commented on issue #6661: [Issue 6658] support multiple authz providers
URL: https://github.com/apache/pulsar/pull/6661#issuecomment-610002085
 
 
   (Due to discuss with @sijie ;))

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [pulsar] KannarFr commented on a change in pull request #6661: [Issue 6658] support multiple authz providers

Posted by GitBox <gi...@apache.org>.
KannarFr commented on a change in pull request #6661: [Issue 6658] support multiple authz providers
URL: https://github.com/apache/pulsar/pull/6661#discussion_r403863465
 
 

 ##########
 File path: pulsar-broker-common/src/main/java/org/apache/pulsar/broker/authorization/AuthorizationService.java
 ##########
 @@ -70,17 +90,38 @@ public AuthorizationService(ServiceConfiguration conf, ConfigurationCacheService
         }
     }
 
+    private static <T> CompletableFuture<T> anyMatch(List<? extends CompletionStage<? extends T>> l,
 
 Review comment:
   The point is: how to get the right auth provider when using authZ provider methods :/.

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [pulsar] KannarFr commented on a change in pull request #6661: [Issue 6658] support multiple authz providers

Posted by GitBox <gi...@apache.org>.
KannarFr commented on a change in pull request #6661: [Issue 6658] support multiple authz providers
URL: https://github.com/apache/pulsar/pull/6661#discussion_r403855179
 
 

 ##########
 File path: pulsar-broker-common/src/main/java/org/apache/pulsar/broker/authorization/AuthorizationService.java
 ##########
 @@ -70,17 +90,38 @@ public AuthorizationService(ServiceConfiguration conf, ConfigurationCacheService
         }
     }
 
+    private static <T> CompletableFuture<T> anyMatch(List<? extends CompletionStage<? extends T>> l,
 
 Review comment:
   So put `String getAuthZClassname();` on `AuthenticationProvider` and use the defined authZ provider depending on the auth provider in `AuthorizationService`. WDYT? @sijie @merlimat 

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [pulsar] KannarFr commented on a change in pull request #6661: [Issue 6658] support multiple authz providers

Posted by GitBox <gi...@apache.org>.
KannarFr commented on a change in pull request #6661: [Issue 6658] support multiple authz providers
URL: https://github.com/apache/pulsar/pull/6661#discussion_r403864172
 
 

 ##########
 File path: pulsar-broker-common/src/main/java/org/apache/pulsar/broker/authorization/AuthorizationService.java
 ##########
 @@ -70,17 +90,38 @@ public AuthorizationService(ServiceConfiguration conf, ConfigurationCacheService
         }
     }
 
+    private static <T> CompletableFuture<T> anyMatch(List<? extends CompletionStage<? extends T>> l,
 
 Review comment:
   Put the auth provider method name to `AuthenticationDataSource`?

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [pulsar] KannarFr commented on a change in pull request #6661: [Issue 6658] support multiple authz providers

Posted by GitBox <gi...@apache.org>.
KannarFr commented on a change in pull request #6661: [Issue 6658] support multiple authz providers
URL: https://github.com/apache/pulsar/pull/6661#discussion_r403870749
 
 

 ##########
 File path: pulsar-broker-common/src/main/java/org/apache/pulsar/broker/authorization/AuthorizationService.java
 ##########
 @@ -70,17 +90,38 @@ public AuthorizationService(ServiceConfiguration conf, ConfigurationCacheService
         }
     }
 
+    private static <T> CompletableFuture<T> anyMatch(List<? extends CompletionStage<? extends T>> l,
 
 Review comment:
   Hum, or not. Anyway, I'm waiting for your point of view.

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [pulsar] KannarFr commented on a change in pull request #6661: [Issue 6658] support multiple authz providers

Posted by GitBox <gi...@apache.org>.
KannarFr commented on a change in pull request #6661: [Issue 6658] support multiple authz providers
URL: https://github.com/apache/pulsar/pull/6661#discussion_r403856174
 
 

 ##########
 File path: pulsar-broker-common/src/main/java/org/apache/pulsar/broker/authorization/AuthorizationService.java
 ##########
 @@ -70,17 +90,38 @@ public AuthorizationService(ServiceConfiguration conf, ConfigurationCacheService
         }
     }
 
+    private static <T> CompletableFuture<T> anyMatch(List<? extends CompletionStage<? extends T>> l,
 
 Review comment:
   ```
       /**
        * @return the authorization methods names supported by this provider
        */
       default Set<String> getAuthZMethodsNames() {
           return Sets.newHashSet("PulsarAuthorizationProvider");
       }
   ```

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [pulsar] KannarFr closed pull request #6661: [Issue 6658] support multiple authz providers

Posted by GitBox <gi...@apache.org>.
KannarFr closed pull request #6661: [Issue 6658] support multiple authz providers
URL: https://github.com/apache/pulsar/pull/6661
 
 
   

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [pulsar] KannarFr commented on issue #6661: [Issue 6658] support multiple authz providers

Posted by GitBox <gi...@apache.org>.
KannarFr commented on issue #6661: [Issue 6658] support multiple authz providers
URL: https://github.com/apache/pulsar/pull/6661#issuecomment-608543857
 
 
   The forms will the same for each authZ provider.

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services