jscheffl opened a new pull request, #59958: URL: https://github.com/apache/airflow/pull/59958
Had a longer research in iterations in our codebase why PR https://github.com/apache/airflow/pull/59819/ created a regression in retrieving variables as discussed in Slack (https://apache-airflow.slack.com/archives/C06K9Q5G2UA/p1767101218465539) Root cause was due to refactoring I changed the default initialization of the fernet_key to be `None` instead of an empty string. This casued the config getter when checking to fallback where `None` was actually passed as fallback but the config value defaults existed and while preparing the default it was attempted to expand variables in the defaults which had in this case not been iniitalized and the string formatter made `"None"` into the defaults which then prevented triggering a regeneration of a new fernet. Which then as using `"None"` (String) as Fernet caused variables to fail encoding in DB. Fix is to use empty string for fernet (as well to be consistent with JWT) as before PR #59819 - to be on the safe side removing the fallback and bad check for missing fernet being passed in config. This PR fixes the regression and adds a test hoping that this does not happen again. -- 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]
