andreyolv commented on issue #18714: URL: https://github.com/apache/airflow/issues/18714#issuecomment-2261130944
I was investigating the possible cause of the problem. I was confused because the name of the [patch_role](https://github.com/apache/airflow/blob/main/airflow/providers/fab/auth_manager/api_endpoints/role_and_permission_endpoint.py#L121-L151) function uses the PUT method in the decorator above, but the behavior is actually PATCH and not PUT, as this function calls the [bulk_sync_roles](https://github.com/apache/airflow/blob/main/airflow/providers/fab/auth_manager/security_manager/override.py#L1396-L1412) function which only adds the roles and to have the PUT behavior it should have [delete_role](https://github.com/apache/airflow/blob/main/airflow/providers/fab/auth_manager/security_manager/override.py#L1472-L1485) inside this function. I don't have in-depth knowledge of Python development, so I could be saying something wrong, I'm just trying to help identify the possible cause and contribute in some way to this issue -- 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]
