ucaeon opened a new pull request, #72790: URL: https://github.com/apache/airflow/pull/72790
`SFTPOperator` and `SFTPSensor` currently do not include their connection IDs in `template_fields`, so Jinja templates used for SFTP connection IDs are not rendered. This PR updates the SFTP provider and: - Adds `ssh_conn_id` to `template_fields` for `SFTPOperator` - Adds `sftp_conn_id` to `template_fields` for `SFTPSensor` - Marks the connection IDs as templated in the corresponding docstrings - Adds tests for templating the connection IDs in both classes `SFTPOperator` uses `ssh_conn_id` because `SFTPHook` inherits from `SSHHook`. This PR keeps the existing argument names unchanged. Tested locally with Breeze: SFTP operator and sensor unit tests — 55 passed, 6 skipped. The HTTP provider part is addressed separately in #72582. related: #35259 --- ##### Was generative AI tooling used to co-author this PR? - [x] Yes — Cursor (Grok) Generated-by: Cursor (Grok) following [the guidelines](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#gen-ai-assisted-contributions) -- 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]
