nicoloboschi opened a new pull request, #19422: URL: https://github.com/apache/pulsar/pull/19422
### Motivation When setting entry filters for namespace/topic there's no check that the entry filters exist and were correctly loaded by the broker. Since the actual entry filters classloaders are loaded in an async way, it's possible to corrupt the entry filters in the policies (it gets back to the broker config) returning OK to the API call. The only way to detect the entry filters is not working it's to look at the broker log (since the GET api call will return the content of the policies). ### Modifications * During the POST method the entry filters are validated. Validated means the entry filter name exists in the broker. * Fixed the GET method with applied=true that wasn't taking into account the global ´AllowOverrideEntryFilters´ broker flag. Also it was mixing sync/async operations ### Documentation <!-- DO NOT REMOVE THIS SECTION. CHECK THE PROPER BOX ONLY. --> - [ ] `doc` <!-- Your PR contains doc changes. --> - [ ] `doc-required` <!-- Your PR changes impact docs and you will update later --> - [x] `doc-not-needed` <!-- Your PR changes do not impact docs --> - [ ] `doc-complete` <!-- Docs have been already added --> -- 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]
