Skip to content

fix: LocalProxy is not mapped warning - #53

Open
landryb wants to merge 2 commits into
main-georchestrafrom
fix/warn-localproxy
Open

fix: LocalProxy is not mapped warning#53
landryb wants to merge 2 commits into
main-georchestrafrom
fix/warn-localproxy

Conversation

@landryb

@landryb landryb commented Aug 26, 2025

Copy link
Copy Markdown
Member

(cherry picked from commit apache/superset@c029b53 via apache/superset#33025)

tested locally, working fine and solving apache/superset#26020, already backported upstream in the 5 branch.

@landryb
landryb requested a review from jeanpommier August 26, 2025 12:10
@landryb

landryb commented Aug 26, 2025

Copy link
Copy Markdown
Member Author

after testing a bit, it seems that upstream commit only fixes the issue for connected users, when loading superset without being connected (eg anonymous?) then you get this annoying warning instead:

Aug 26 14:48:11 WARNING:root:Failed to add user to db session: Class 'werkzeug.local.LocalProxy' is not mapped

maybe the logging.warning call should be neuteured in our fork, or the underlying issue actually fixed ? or that's something specific to our integration that doesnt do the right thing when there's no user ?

@landryb

landryb commented Aug 26, 2025

Copy link
Copy Markdown
Member Author

@jeanpommier btw i still get the warning, even with https://github.com/georchestra/superset/blame/main/config/superset/GeorchestraCustomizations.py#L361 - i dunno if the doesnt work right now comment is related ..

@landryb

landryb commented Aug 26, 2025

Copy link
Copy Markdown
Member Author

with 1af1746 i dont get warnings anymore - suggested here, hoping for feedback.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants