neuromantik33 commented on PR #30549:
URL: https://github.com/apache/superset/pull/30549#issuecomment-2962176243

   Hello. This PR breaks are existing usage of RLS (thank heavens one for our 
campus users notified us). This is our input string:
   ```sql
   campus_id IN UNNEST( {{ cache_key_wrapper(current_user_campus_ids()) }} )
   ```
   which doesn't match using the pattern 
   `cache_key_wrapper\([^()]*\)`
   
   , but does using the old one (v4.0.2)
   `cache_key_wrapper\(.*\)`.
   
   Is there somewhere in the docs on how we can manage this? The function we 
use looks as follows:
   
   ```python
   def current_user_campus_ids() -> list[int]:
       """
       Get the list of campus ids associated with the current user
       """
       user: FortyTwoUser = g.user
       return user.campus_ids()
   ```
   and is populated using the oauth2 jwt from keycloak. Any help would be much 
appreciated.


-- 
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]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to