Samin061 commented on PR #70000:
URL: https://github.com/apache/airflow/pull/70000#issuecomment-5030778421

   Good call on both.
   
   Added a breaking-changes note at the top of 
`providers/http/docs/changelog.rst` covering exactly that case: if you were 
relying on the connection's headers reaching the redirect target (a CDN or 
object store that needs the same key), the options are to point the connection 
at a host that matches the final destination, or set `allow_redirects=False` in 
`extra_options` and issue the second request yourself.
   
   Opened #70164 for the async side, and left a pointer to it next to the 
header handling in `HttpAsyncHook.config` so it survives after this merges. 
`aiohttp` has no per-redirect callback equivalent to 
`requests.Session.rebuild_auth`, so that fix means walking the redirect chain 
manually rather than subclassing, which is why it isn't in here.


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to