jedcunningham opened a new pull request, #39246:
URL: https://github.com/apache/airflow/pull/39246

   Closes: #38836 
   Alternative to #38876 
   
   Do the encryption in the migration itself, and fix support for offline 
migrations as well.
   
   The offline up migration won't actually encrypt the trigger kwargs as there 
isn't a safe way to accomplish that, so the decryption processes checks and 
short circuits if it isn't encrypted.
   
   The offline down migration will now print out a warning that the offline 
migration will fail if there are any running triggers. I think this is the best 
we can do for that scenario (and folks willing to do offline migrations will 
hopefully be able to understand the situation).
   
   This also solves the "encrypting the already encrypted kwargs" bug in 2.9.0.
   
   


-- 
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]

Reply via email to