[GitHub] [superset] nytai commented on pull request #23280: fix: Redirect to login on unauthorised access for Dashboard.

["nytai (via GitHub)" <gi...@apache.org>]

nytai commented on PR #23280:
URL: https://github.com/apache/superset/pull/23280#issuecomment-1459439103

   @sfirke the reason for that is when using a `Public` role then there is a current user, it's just stubbed. It's a bit tricky because the Public role exposes content to the public, not logged in user. How do you handle that? It's hard to know wether the user doesn't have access because they aren't logged in or because you don't want this content exposed. 


-- 
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: notifications-unsubscribe@superset.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org