[GitHub] [airflow] pauldalewilliams commented on pull request #24512: Convert sftp hook to use paramiko instead of pysftp

[GitBox <gi...@apache.org>]

pauldalewilliams commented on PR #24512:
URL: https://github.com/apache/airflow/pull/24512#issuecomment-1158945607

   One other note I forgot to include: the default for `no_host_key_check` used to be `true` for `SFTPHook` but it will now default to `false` since that is the default for `SSHHook`.  That's not a breaking change but those relying on it to default to `true` will need to explicitly add it to the extra parameters on the connection.
   
   The primary motivation for doing this was to ensure that the SFTP connections can support all of the parameters supported by `SSHHook`. Because `pysftp` did not support passing all those parameters to `paramiko` I figured this was the best way to get that consistency.


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