abhishekmauryaKsolves opened a new pull request, #73597:
URL: https://github.com/apache/airflow/pull/73597
SFTPHookAsync._parse_extras() compared the already-expanded self.known_hosts
against the un-expanded default_known_hosts literal, so the comparison was
always true and a connection's known_hosts extra could override even an
explicit constructor path. Fixed to expand default_known_hosts before
comparing, matching SSHHookAsync's behavior.
Added a regression test that constructs SFTPHookAsync with an explicit
known_hosts path, parses a connection with a different known_hosts extra,
and asserts the explicit constructor value is retained.
closes: #73584
<!-- SPDX-License-Identifier: Apache-2.0
https://www.apache.org/licenses/LICENSE-2.0 -->
---
##### Was generative AI tooling used to co-author this PR?
- [X] Yes (Claude, Anthropic)
<!--
Generated-by: Claude (Anthropic) following [the
guidelines](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#gen-ai-assisted-contributions)
-->
---
* Read the **[Pull Request
Guidelines](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#pull-request-guidelines)**
for more information. Note: commit author/co-author name and email in commits
become permanently public when merged.
--
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]
