nandorKollar commented on PR #4225: URL: https://github.com/apache/polaris/pull/4225#issuecomment-4638057480
With this sanitisation logic, we can filter out AttributeKeys but can't filter out a certain property of an object right? So for example, we can define that we don't allow event listeners to receive `UpdatePrincipalRequest` (with `update_catalog_request` key), but we can filter out just some properties of this request right? I think it is fine, just would like to understand the limitations, and I'm wondering that the scenario is in fact relevant or not at all. The overall approach looks good to me. I think we should also mention this new feature in the release notes. -- 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]
