Joffreybvn commented on PR #35591: URL: https://github.com/apache/airflow/pull/35591#issuecomment-1825016420
Thanks for the detailed answer ! I won't go for proxy settings and extra parameters in this PR. Just for info, adding a tool like [forwarder](https://github.com/saucelabs/forwarder) solve globally all proxy configuration issues. And disabling ssl "verify" can be done via the [`CURL_CA_BUNDLE` trick]() instead of adding parameters to control that in the Connection. --- For the rest, currently this PR: - Allows to configure the Auth class from the Connection - Allows to pass extra parameters to instantiate the Auth class from the Connection There is no breaking change in Airflow. It may be a breaking change for user relying on the previous logic of the property (see below's code-review). But as the property was introduced recently, that won't break _many_ users' worflows. -- 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]
