Vastly simplify the auth hooks interface

Author: mdellweg

pulp-glue/src/pulp_glue/common/authentication.py

@@ -42,14 +42,10 @@ def can_complete(
         # This covers the case where `[]` allows for no auth at all.
         return True
 
-    async def auth_success_hook(
-        self, proposal: dict[str, list[str]], security_schemes: dict[str, dict[str, t.Any]]
-    ) -> None:
+    async def auth_success_hook(self, **kwargs: t.Any) -> None:
         pass
 
-    async def auth_failure_hook(
-        self, proposal: dict[str, list[str]], security_schemes: dict[str, dict[str, t.Any]]
-    ) -> None:
+    async def auth_failure_hook(self, **kwargs: t.Any) -> None:
         pass
 
     async def http_basic_credentials(self) -> tuple[bytes, bytes]:

pulp-glue/src/pulp_glue/common/openapi.py

@@ -726,17 +726,9 @@ def call(
                 response = self._send_request(request)
 
             if response.status_code >= 200 and response.status_code < 300:
-                asyncio.run(
-                    self._auth_provider.auth_success_hook(
-                        proposal, self.api_spec["components"]["securitySchemes"]
-                    )
-                )
+                asyncio.run(self._auth_provider.auth_success_hook())
             elif response.status_code == 401:
-                asyncio.run(
-                    self._auth_provider.auth_failure_hook(
-                        proposal, self.api_spec["components"]["securitySchemes"]
-                    )
-                )
+                asyncio.run(self._auth_provider.auth_failure_hook())
 
         self._log_response(response)
         return self._parse_response(method_spec, response)

src/pulp_cli/generic.py

@@ -281,9 +281,7 @@ def _save_password_to_storage(self) -> None:
                 collection = secretstorage.get_default_collection(connection)
                 collection.create_item("Pulp CLI", secret_attr, password, replace=True)
 
-    async def auth_success_hook(
-        self, proposal: dict[str, list[str]], security_schemes: dict[str, dict[str, t.Any]]
-    ) -> None:
+    async def auth_success_hook(self, **kwargs: t.Any) -> None:
         if SECRET_STORAGE and self._password_in_secretstorage is False:
             await asyncio.get_running_loop().run_in_executor(None, self._save_password_to_storage)
         self._password_in_secretstorage = None
@@ -305,9 +303,7 @@ def _remove_password_from_storage(self) -> None:
                     item.delete()
         self.password = None
 
-    async def auth_failure_hook(
-        self, proposal: dict[str, list[str]], security_schemes: dict[str, dict[str, t.Any]]
-    ) -> None:
+    async def auth_failure_hook(self, **kwargs: t.Any) -> None:
         if SECRET_STORAGE and self._password_in_secretstorage is True:
             await asyncio.get_running_loop().run_in_executor(
                 None, self._remove_password_from_storage