jscheffl commented on code in PR #41777:
URL: https://github.com/apache/airflow/pull/41777#discussion_r1735316225
##########
airflow/auth/managers/base_auth_manager.py:
##########
@@ -443,3 +430,19 @@ def security_manager(self) -> AirflowSecurityManagerV2:
from airflow.www.security_manager import AirflowSecurityManagerV2
return AirflowSecurityManagerV2(self.appbuilder)
+
+ @staticmethod
+ def get_cli_commands() -> list[CLICommand]:
+ """
+ Vends CLI commands to be included in Airflow CLI.
+
+ Override this method to expose commands via Airflow CLI to manage this
auth manager.
+ """
+ return []
+
+ def get_api_endpoints(self) -> None | Blueprint:
+ """Return API endpoint(s) definition for the auth manager."""
+ return None
+
+ def register_views(self) -> None:
Review Comment:
FYI @bbovenzi or @pierrejeambrun this smells a bit like the idea of AIP-68
but for another purpose - do you have an idea how to expose "views" w/o
destroying the future UI concept in React? (w/o having the new plugin mechanism
in place today and and not adding a new binding to FAB?
--
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]
