landryb commented on PR #33025: URL: https://github.com/apache/superset/pull/33025#issuecomment-3224136781
Fwiw, after testing, i'm not sure the PR fixes the actual underlying issue. with the PR im now getting `WARNING:root:Failed to add user to db session: Class 'werkzeug.local.LocalProxy' is not mapped` messages when visiting superset as an anonymous/unconnected user, in an infrastructure using `REMOTE_USER` after looking at the code and adding some debug printfs, the `actual_user` object is a `flask_login.mixins.AnonymousUserMixin` that we try to serialize into the database logging, via apparently that 'werkzeug localproxy' intermediate class. And after searching a bit on the interwebs, the _class is not mapped_ message comes from the underlying sqlalchemy bits. when the user is connected, that block doesn't trigger and there's no warning, since `user_id` isn't `None`. -- 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-unsubscr...@superset.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: notifications-unsubscr...@superset.apache.org For additional commands, e-mail: notifications-h...@superset.apache.org
