dwreeves opened a new pull request, #62378: URL: https://github.com/apache/airflow/pull/62378
Adds support for private keys that are not base64 encoded. This was available prior to the Snowflake provider 6.3.0 release, then it was changed. There is discussion of this change in #47003 and its corresponding PR. I did not see anyone float the idea of supporting _both_ and a switch was made. It is argued in the PR that this is not considered a breaking change, rather it is a bug. But in my experience, this is both a breaking change and also not a bug. I use the AWS Secrets Manager backend, not the GUI, and there is no problem on my end storing plain text passwords inside of the AWS Secrets Manager, nor do I have any issues encoding newlines. Migrating to ≥ 6.3.0 would involve significant overhead on my end in not only rotating passwords but changing how they are managed, without a clear migration path which eliminates downtime (precisely because both methods of storing private keys are not supported simultaneously). This is coming nearly a year later, so perhaps a bit later than it should be if it is to be accepted. But I'm only just doing these migrations right now. I have an open issue #62359 where this can be discussed further. * closes: #62359 --- ##### Was generative AI tooling used to co-author this PR? - [ ] Yes (please specify the tool below) -- 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]
