benjreinhart commented on pull request #14702:
URL: https://github.com/apache/superset/pull/14702#issuecomment-844457797


   > For public users with no user ID, we may need to tweak the token 
invalidation logic, depending on the session logout handling (needs testing).
   
   We'll also need to either:
   
   1. Make all the chart/dashboard query logic behave when `g.user` is not set 
(today, it assumes it will always be there), or...
   2. Set g.user to a mock user in Celery tasks when there is no user_id 
present and ensure all code paths can respect this mock user object


-- 
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:
[email protected]



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

Reply via email to