Aditya-kumar-yadav commented on PR #75215: URL: https://github.com/apache/airflow/pull/75215#issuecomment-6294019595
Thanks for reviewing @Eason09053360! You are completely right — `Connection.get_uri()` percent-encodes the password, so `@` in the password becomes `%40`. I have updated the PR accordingly: 1. Updated the tests to test `Connection(...).get_uri()` directly across the different credential configurations (user+password, username-only, password-only token, empty password, no credentials). 2. Updated the PR description and commit message to drop the leak claim and focus on preserving credential structure (avoiding artificial `:***` injection for username-only/token connections). Pushed the updated commit! -- 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]
