VVildVVolf commented on PR #31203: URL: https://github.com/apache/airflow/pull/31203#issuecomment-1545371020
@potiuk , > You can see example changes that needed to be ported in past PRs that were upgrading FAB BTW. Yes, I am already using https://github.com/apache/airflow/pull/29766/files as a reference, thanks! > Did you check if there are no changes in security classes folowing this comment? Mostly I was focused on checking if upgrade of FAB does help or not with the issue https://github.com/apache/airflow/issues/31156 . Good news that upgrading to 4.3.1 helps (but hard to be covered by Unit test, investigating if it is possible). The review of `airflow/www/fab_security` is in progress - I need to find out some details. P.S. Just for history maybe it will help someone: to update FAB locally helped: - some manipulations with [docker's ARGs like AIRFLOW_SOURCES_FROM](https://github.com/apache/airflow/blob/main/Dockerfile) - `breeze ci-image build --upgrade-to-newer-dependencies ` -- 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]
