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


   > > 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).
   
   I have used request.blueprint. Maybe we should refactor code later on and 
not hard-code the URL. 
   Let me know if this is what you expected.


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