[GitHub] [pulsar] michaeljmarshall commented on pull request #11777: [Authorization] AuthorizationService should use provider's canLookupAsync method

2021-09-01 Thread GitBox
michaeljmarshall commented on pull request #11777: URL: https://github.com/apache/pulsar/pull/11777#issuecomment-909418006 > @michaeljmarshall if it is a major change or affects how to use this feature, maybe you add a note about the **previous usage** and **current changes** [here](https

[GitHub] [pulsar] michaeljmarshall commented on pull request #11777: [Authorization] AuthorizationService should use provider's canLookupAsync method

2021-08-31 Thread GitBox
michaeljmarshall commented on pull request #11777: URL: https://github.com/apache/pulsar/pull/11777#issuecomment-909418006 > @michaeljmarshall if it is a major change or affects how to use this feature, maybe you add a note about the **previous usage** and **current changes** [here](https

[GitHub] [pulsar] michaeljmarshall commented on pull request #11777: [Authorization] AuthorizationService should use provider's canLookupAsync method

2021-08-26 Thread GitBox
michaeljmarshall commented on pull request #11777: URL: https://github.com/apache/pulsar/pull/11777#issuecomment-906780067 @Anonymitaet, @eolivelli, @sijie - how do we document changes that correct existing behavior? This change will _only_ affect users with custom `AuthorizationProvider`

[GitHub] [pulsar] michaeljmarshall commented on pull request #11777: [Authorization] AuthorizationService should use provider's canLookupAsync method

2021-08-26 Thread GitBox
michaeljmarshall commented on pull request #11777: URL: https://github.com/apache/pulsar/pull/11777#issuecomment-906746835 /pulsarbot run-failure-checks -- 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

[GitHub] [pulsar] michaeljmarshall commented on pull request #11777: [Authorization] AuthorizationService should use provider's canLookupAsync method

2021-08-25 Thread GitBox
michaeljmarshall commented on pull request #11777: URL: https://github.com/apache/pulsar/pull/11777#issuecomment-905893898 /pulsarbot run-failure-checks -- 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