ephraimbuddy commented on pull request #15295: URL: https://github.com/apache/airflow/pull/15295#issuecomment-816634247
> > The reason is that apart from the REST API we would not have the need of modifying the 'flask' session or stopping its creation in another module. > > RIght now we don't. But modifying the session interface is still a global override, and doing it inside `api_connextion` would waste future contributors time debugging why their override doesn't work (because it's overridden again during `init_api_connexion()`), or mysteriously breaks the API (because their session interface overrides the setup in `init_api_connexion()`). Putting the override in `create_app()` helps prevent that. Agreed! Thanks. I will work out a solution on the security manager. -- 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]
