[API] /connections/test: Use stored password for saved connections and fix DBT hook URL#59917
[API] /connections/test: Use stored password for saved connections and fix DBT hook URL#59917anishgirianish wants to merge 2 commits intoapache:mainfrom
Conversation
There was a problem hiding this comment.
@anishgirianish Why did you close this PR, do you plan to work on a followup? I think this still needs to be fixed.
|
Ah maybe because there was #59643 opened already. Cool |
|
Hi @pierrejeambrun Apologies for replying late. thank you so much for your reply. I figured only after opening this pr that there was already a Pr open with similar approach as this. Also going through discussion I saw desirable way of implementing it and thought closing this out. I am very intrested to work on a proposal for the discussed approach in #59643 if work has not been started on that. If so would like to request you for the assignment thank you. |
|
@anishgirianish You don't need any assignment. You can just start working on a PR, I'd be glad to review it for the short term fix so people have a workaround while we work on the long term fix option |
This PR fixes the /connections/test route in Airflow v3 to handle masked passwords correctly and ensures DBT hooks test connections without errors.
What/Why
How
Impact
closes: #58941
related: #58941
^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named
{pr_number}.significant.rstor{issue_number}.significant.rst, in airflow-core/newsfragments.