[PR] Fix Execution API refresh token [airflow]

["pierrejeambrun (via GitHub)" <gi...@apache.org>]

pierrejeambrun opened a new pull request, #58782:
URL: https://github.com/apache/airflow/pull/58782

   Closes: https://github.com/apache/airflow/issues/58775
   
   That's due to the recent change in FastAPI and how they handle responses (and fixes a bug for streaming responses). 
   
   Basically headers set after the `yield` of the dependency are not reflected in the actual response. 
   
   As a solution, similarly to the core API we now use a middleware instead to achieve the same.


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

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] Fix Execution API refresh token [airflow]

["pierrejeambrun (via GitHub)" <gi...@apache.org>]

pierrejeambrun commented on PR #58782:
URL: https://github.com/apache/airflow/pull/58782#issuecomment-3586877645

   Tested locally, this fixes the issue for me.


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

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] Fix Execution API refresh token [airflow]

["vincbeck (via GitHub)" <gi...@apache.org>]

vincbeck merged PR #58782:
URL: https://github.com/apache/airflow/pull/58782


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

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org