uranusjr commented on pull request #15295:
URL: https://github.com/apache/airflow/pull/15295#issuecomment-817958781


   > if user decides to configure it, then the user should also configure 
session interface
   
   Sounds good to me!
   
   ----
   
   Instead of detecting the `/api/` prefix, would it be better to rely on 
Flask’s routing for detection instead? IIRC API views are grouped in Blueprints 
already, so maybe `request.blueprint` would be useful. I’m not strong on this 
though; hard-coding URLs is usually not a good idea, but Airflow is already 
doing that in may places anyway (eh).


-- 
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]


Reply via email to