You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@airflow.apache.org by "Michalosu (via GitHub)" <gi...@apache.org> on 2023/08/24 15:00:43 UTC

[GitHub] [airflow] Michalosu commented on pull request #33690: SECURITY_MANAGER_CLASS should be a referrence to class, not a string

Michalosu commented on PR #33690:
URL: https://github.com/apache/airflow/pull/33690#issuecomment-1691850708

   It would be good to mentioned it in release note of 2.7.0 as this is kinda breaking change if you have own implementation of `AirflowSecurityManager`. Code has been change that you can't use `FAB_SECURITY_MANAGER_CLASS` but docs were still pointing to that and with the wrong example of value. 
   
   Good that I know what happened after upgrade to 2.7.0 and created this PR for others if someone else will see this problem - "Boy scout rule" 


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org