Pedrinhonitz opened a new pull request, #72348:
URL: https://github.com/apache/airflow/pull/72348
<!-- SPDX-License-Identifier: Apache-2.0
https://www.apache.org/licenses/LICENSE-2.0 -->
# Resume
Fixes #72318. In the UI's "Test Connection" feature, unused host/port fields
were stored as NULL (None) in the database, while the UI sent an empty string
(""). This discrepancy caused the system to treat it as a new destination, skip
the password/extra merge, and pass `private_key_file="***"` to the hook. Now,
blank host/port fields are treated as the same destination, and `***` is not
considered a new credential.
---
# Issues
* closes: #72318
---
##### Was generative AI tooling used to co-author this PR?
<!--
If generative AI tooling has been used in the process of authoring this PR,
please
change below checkbox to `[X]` followed by the name of the tool, uncomment
the "Generated-by".
-->
- [x] Yes (please specify the tool below)
Generated-by: Cursor IA IDLE following [the
guidelines](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#gen-ai-assisted-contributions)
, to assist with understanding and documentation.
--
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]
