Do not treat handlers actions that have not been implemented as a dispatch failure

CHANGES.rst

@@ -11,6 +11,7 @@ Changes
 * Pin ``watchdog>=5`` to employ typing fixes.
 * Pin ``requests>=2.32.3`` to fix security vulnerability.
 * Pin ``setuptools>=70.0.0`` to fix security vulnerability.
+* Do not treat handlers actions that have not been implemented as a dispatch failure.
 
 `2.4.0 <https://github.com/Ouranosinc/cowbird/tree/2.4.0>`_ (2024-07-09)
 ------------------------------------------------------------------------------------

cowbird/handlers/impl/catalog.py

@@ -35,10 +35,10 @@ def user_deleted(self, user_name: str) -> None:
         Monitoring().unregister(self._user_workspace_dir(user_name), self)
 
     def permission_created(self, permission: Permission) -> None:
-        raise NotImplementedError
+        LOGGER.info("Event [permission_created] for handler [%s] is not implemented", self.name)
 
     def permission_deleted(self, permission: Permission) -> None:
-        raise NotImplementedError
+        LOGGER.info("Event [permission_deleted] for handler [%s] is not implemented", self.name)
 
     @staticmethod
     def get_instance() -> Optional["Catalog"]:
@@ -74,4 +74,4 @@ def on_modified(self, path: str) -> None:
 
     def resync(self) -> None:
         # FIXME: this should be implemented in the eventual task addressing the resync mechanism.
-        raise NotImplementedError
+        LOGGER.warning("Event [resync] for handler [%s] is not implemented but should be in the future", self.name)

cowbird/handlers/impl/geoserver.py

@@ -388,7 +388,7 @@ def on_modified(self, path: str) -> None:
 
     def resync(self) -> None:
         # FIXME: this should be implemented in the eventual task addressing the resync mechanism.
-        raise NotImplementedError
+        LOGGER.warning("Event [resync] for handler [%s] is not implemented but should be in the future", self.name)
 
     @staticmethod
     def _is_permission_update_required(effective_permissions: List[JSON],

cowbird/handlers/impl/magpie.py

@@ -290,10 +290,12 @@ def get_group_permissions_by_res_id(self, grp: str, res_id: int, effective: bool
         return resp.json()
 
     def user_created(self, user_name: str) -> None:
-        raise NotImplementedError
+        LOGGER.warning("Event [user_created] for handler [%s] is not implemented since Magpie "
+                       "has already created the user.", self.name)
 
     def user_deleted(self, user_name: str) -> None:
-        raise NotImplementedError
+        LOGGER.warning("Event [user_deleted] for handler [%s] is not implemented since Magpie "
+                       "has already deleted the user.", self.name)
 
     def permission_created(self, permission: Permission) -> None:
         self.permissions_synch.create_permission(permission)
@@ -303,7 +305,7 @@ def permission_deleted(self, permission: Permission) -> None:
 
     def resync(self) -> None:
         # FIXME: this should be implemented in the eventual task addressing the resync mechanism.
-        raise NotImplementedError
+        LOGGER.warning("Event [resync] for handler [%s] is not implemented but should be in the future", self.name)
 
     def create_permissions(self, permissions_data: List[PermissionConfigItemType]) -> None:
         """

cowbird/handlers/impl/nginx.py

@@ -3,6 +3,10 @@
 from cowbird.handlers.handler import Handler
 from cowbird.permissions_synchronizer import Permission
 from cowbird.typedefs import SettingsType
+from cowbird.utils import get_logger
+
+
+LOGGER = get_logger(__name__)
 
 
 class Nginx(Handler):
@@ -21,17 +25,17 @@ def __init__(self, settings: SettingsType, name: str, **kwargs: Any) -> None:
         super(Nginx, self).__init__(settings, name, **kwargs)
 
     def user_created(self, user_name: str) -> None:
-        raise NotImplementedError
+        LOGGER.info("Event [user_created] for handler [%s] is not implemented", self.name)
 
     def user_deleted(self, user_name: str) -> None:
-        raise NotImplementedError
+        LOGGER.info("Event [user_deleted] for handler [%s] is not implemented", self.name)
 
     def permission_created(self, permission: Permission) -> None:
-        raise NotImplementedError
+        LOGGER.info("Event [permission_created] for handler [%s] is not implemented", self.name)
 
     def permission_deleted(self, permission: Permission) -> None:
-        raise NotImplementedError
+        LOGGER.info("Event [permission_deleted] for handler [%s] is not implemented", self.name)
 
     def resync(self) -> None:
         # FIXME: this should be implemented in the eventual task addressing the resync mechanism.:
-        raise NotImplementedError
+        LOGGER.warning("Event [resync] for handler [%s] is not implemented but should be in the future", self.name)

cowbird/handlers/impl/thredds.py

@@ -3,6 +3,10 @@
 from cowbird.handlers.handler import Handler
 from cowbird.permissions_synchronizer import Permission
 from cowbird.typedefs import SettingsType
+from cowbird.utils import get_logger
+
+
+LOGGER = get_logger(__name__)
 
 
 class Thredds(Handler):
@@ -21,17 +25,17 @@ def __init__(self, settings: SettingsType, name: str, **kwargs: Any) -> None:
         super(Thredds, self).__init__(settings, name, **kwargs)
 
     def user_created(self, user_name: str) -> None:
-        raise NotImplementedError
+        LOGGER.info("Event [user_created] for handler [%s] is not implemented", self.name)
 
     def user_deleted(self, user_name: str) -> None:
-        raise NotImplementedError
+        LOGGER.info("Event [user_deleted] for handler [%s] is not implemented", self.name)
 
     def permission_created(self, permission: Permission) -> None:
-        raise NotImplementedError
+        LOGGER.info("Event [permission_created] for handler [%s] is not implemented", self.name)
 
     def permission_deleted(self, permission: Permission) -> None:
-        raise NotImplementedError
+        LOGGER.info("Event [permission_deleted] for handler [%s] is not implemented", self.name)
 
     def resync(self) -> None:
         # FIXME: this should be implemented in the eventual task addressing the resync mechanism.
-        raise NotImplementedError
+        LOGGER.warning("Event [resync] for handler [%s] is not implemented but should be in the future", self.name)