Samin061 commented on PR #70000: URL: https://github.com/apache/airflow/pull/70000#issuecomment-6001567235
Makes sense, closing this in favour of #72165 and #72398. The middleware is the better fit for the async side since it leaves aiohttp's own redirect handling alone, which the walker here had to re-implement. One thing from the earlier review here that I don't see in either yet: a changelog line on what to do if you relied on the old behaviour (point the connection at the final host, or set `allow_redirects=False` in `extra_options` and make the second request yourself). --- Drafted-by: Claude (Fable 5.1) (no human review before posting) -- 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]
