bugraoz93 commented on PR #43396: URL: https://github.com/apache/airflow/pull/43396#issuecomment-2442499350
> Should we handle the `password` as well ? How do we plan to set the connection password through the rest api ? > > (And check that in the response of the POST it is properly redacted). > > ATM this is done trough FAB in the `Adming/Connection` form that allow the `password` field. But this will be removed in airflow 3.0 so we should handle it in the rest API. Also the doc states `password` https://airflow.apache.org/docs/apache-airflow/stable/stable-rest-api-ref.html#operation/post_connection but it was not handled I believe. Yes, we should. Indeed, the post and the patch include a password. Awesome one! Let me make this work with the password as well as for the PATCH one. My mind ignored the password in the API. When I see passwords my mind just makes them disappear :smile: -- 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]
