mistercrunch commented on issue #4669: [sqllab] Using app context for processing Jinja template in async mode URL: https://github.com/apache/incubator-superset/pull/4669#issuecomment-376430494 It's a bit hard to read the PR since Github doesn't highlight the right lines at all here. Crazy how an indent completely confuses their diffing function. But even with the Flask context manager you still won't get the user info I'm guessing as the user is not authenticated on the celery side of things. I guess it just fixes that error, but probably does not impersonate the right user deep down the call stack. There should not be an app context at that point, so my thought is we have to clean up the reliance on `g.user` and pass it down the stack instead.
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services
