davlum commented on pull request #15013: URL: https://github.com/apache/airflow/pull/15013#issuecomment-810443658
@dstandish If you read the description of PR you can see an example of `get_uri` failing. The fact is that connection URI is a lossy format and is required by Vault integration. As somebody who manages several Airflow instances and had to migrate all the tenants connections into Vault this was a breaking change as users who did not have unnested JSON had their connection broken when calling `.get_uri()` on it. It was then required to go in and manually change the application code for tenants that used their `extra` parameters to use the appropriate format. This is not a good workflow. -- 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. For queries about this service, please contact Infrastructure at: [email protected]
