jedcunningham commented on pull request #20346: URL: https://github.com/apache/airflow/pull/20346#issuecomment-997073751
I'm just saying that, regardless of what security manager is in use, global `can_edit` should be expected to give edit on all DAGs. We shouldn't assume all things will use `get_editable_dags_ids`. Maybe we add a new `can_edit_all_dags` as a consistent way to check for that (and that assumption-breaking security managers can always return as false)? It'll eventually short-circuit, but in the worst case scenario what impact do we have by not using the more efficient short circuit? -- 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]
