commonground-api-common 1.13.3__py3-none-any.whl → 2.0.0__py3-none-any.whl

vng_api_common/notifications/migrations/0011_remove_subscription_config_and_more.py

@@ -0,0 +1,23 @@
+# Generated by Django 5.1.2 on 2024-10-25 14:07
+
+from django.db import migrations
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ("notifications", "0010_auto_20220704_1419"),
+    ]
+
+    operations = [
+        migrations.RemoveField(
+            model_name="subscription",
+            name="config",
+        ),
+        migrations.DeleteModel(
+            name="NotificationsConfig",
+        ),
+        migrations.DeleteModel(
+            name="Subscription",
+        ),
+    ]

vng_api_common/oas.py

@@ -1,8 +1,5 @@
 """
 Utility module for Open API Specification 3.0.x.
-
-This should get merged into gemma-zds-client, but some heavy refactoring is
-needed for that.
 """
 
 from typing import Union

vng_api_common/routers.py

@@ -8,7 +8,7 @@ class APIRootView(_APIRootView):
     permission_classes = ()
 
 
-class ZDSNestedRegisteringMixin:
+class NestedRegisteringMixin:
     _nested_router = None
 
     def __init__(self, *args, **kwargs):
@@ -47,11 +47,11 @@ class ZDSNestedRegisteringMixin:
             )
 
 
-class NestedSimpleRouter(ZDSNestedRegisteringMixin, routers.NestedSimpleRouter):
+class NestedSimpleRouter(NestedRegisteringMixin, routers.NestedSimpleRouter):
     pass
 
 
-class DefaultRouter(ZDSNestedRegisteringMixin, routers.DefaultRouter):
+class DefaultRouter(NestedRegisteringMixin, routers.DefaultRouter):
     APIRootView = APIRootView
 
 

vng_api_common/tests/schema.py

@@ -5,6 +5,8 @@ from urllib.parse import urlparse
 from django.conf import settings
 
 import yaml
+from requests_mock import Mocker
+from zgw_consumers_oas.schema_loading import read_schema
 
 DEFAULT_PATH_PARAMETERS = {"version": "1"}
 
@@ -69,3 +71,13 @@ def get_validation_errors(response, field, index=0):
             return error
 
         i += 1
+
+
+def mock_service_oas_get(
+    mock: Mocker, url: str, service: str, oas_url: str = ""
+) -> None:
+    if not oas_url:
+        oas_url = f"{url}schema/openapi.yaml?v=3"
+
+    content = read_schema(service)
+    mock.get(oas_url, content=content)

vng_api_common/utils.py

@@ -13,9 +13,6 @@ from django.utils.encoding import smart_str
 from django.utils.module_loading import import_string
 
 from rest_framework.utils import formatting
-from zds_client.client import ClientError
-
-from .client import get_client
 
 try:
     from djangorestframework_camel_case.util import (
@@ -186,25 +183,6 @@ def get_uuid_from_path(path: str) -> str:
     return uuid_str
 
 
-def request_object_attribute(
-    url: str, attribute: str, resource: Union[str, None] = None
-) -> str:
-    client = get_client(url)
-
-    try:
-        result = client.retrieve(resource, url=url)[attribute]
-    except (ClientError, KeyError) as exc:
-        logger.warning(
-            "%s was retrieved from %s with the %s: %s",
-            attribute,
-            url,
-            exc.__class__.__name__,
-            exc,
-        )
-        result = ""
-    return result
-
-
 def generate_unique_identification(instance: models.Model, date_field_name: str):
     model = type(instance)
     model_name = getattr(model, "IDENTIFICATIE_PREFIX", model._meta.model_name.upper())

vng_api_common/validators.py

@@ -11,10 +11,8 @@ from django.utils.deconstruct import deconstructible
 from django.utils.module_loading import import_string
 from django.utils.translation import gettext_lazy as _
 
-import requests
 from rest_framework import serializers, validators
 
-from .client import get_client
 from .constants import RSIN_LENGTH
 from .oas import fetcher, obj_has_shape
 
@@ -221,42 +219,6 @@ class InformatieObjectUniqueValidator(validators.UniqueTogetherValidator):
         super().__call__(attrs)
 
 
-class ObjectInformatieObjectValidator:
-    """
-    Validate that the INFORMATIEOBJECT is linked already in the DRC.
-    """
-
-    message = _(
-        "Het informatieobject is in het DRC nog niet gerelateerd aan dit object."
-    )
-    code = "inconsistent-relation"
-    requires_context = True
-
-    def __call__(self, informatieobject: str, serializer):
-        object_url = serializer.context["parent_object"].get_absolute_api_url(
-            self.request
-        )
-
-        # dynamic so that it can be mocked in tests easily
-        client = get_client(informatieobject)
-
-        try:
-            oios = client.list(
-                "objectinformatieobject",
-                query_params={
-                    "informatieobject": informatieobject,
-                    "object": object_url,
-                },
-            )
-        except requests.HTTPError as exc:
-            raise serializers.ValidationError(
-                exc.args[0], code="relation-validation-error"
-            ) from exc
-
-        if len(oios) == 0:
-            raise serializers.ValidationError(self.message, code=self.code)
-
-
 @deconstructible
 class UntilNowValidator:
     """

vng_api_common/views.py

@@ -1,6 +1,7 @@
 import logging
 import os
 from collections import OrderedDict
+from typing import Optional
 
 from django.apps import apps
 from django.conf import settings
@@ -13,7 +14,8 @@ import requests
 from rest_framework import exceptions as drf_exceptions, status
 from rest_framework.response import Response
 from rest_framework.views import exception_handler as drf_exception_handler
-from zds_client import ClientError
+
+from vng_api_common.client import Client, ClientError
 
 from . import exceptions
 from .compat import sentry_client
@@ -128,12 +130,20 @@ def _test_ac_config() -> list:
     auth_config = AuthorizationsConfig.get_solo()
 
     # check if AC auth is configured
-    ac_client = AuthorizationsConfig.get_client()
-    has_ac_auth = ac_client.auth is not None
+    ac_client: Optional[Client] = AuthorizationsConfig.get_client()
+    has_ac_auth = ac_client.auth is not None if ac_client else False
 
     checks = [
         (_("Type of component"), auth_config.get_component_display(), None),
-        (_("AC"), auth_config.api_root, auth_config.api_root.endswith("/")),
+        (
+            _("AC"),
+            (
+                auth_config.authorizations_api_service.api_root
+                if ac_client
+                else _("Missing")
+            ),
+            bool(ac_client),
+        ),
         (
             _("Credentials for AC"),
             _("Configured") if has_ac_auth else _("Missing"),
@@ -145,18 +155,17 @@ def _test_ac_config() -> list:
     if has_ac_auth:
         error = False
 
+        client_id = ac_client.auth.service.client_id
+
         try:
-            ac_client.list(
-                "applicatie", query_params={"clientIds": ac_client.auth.client_id}
+            response: requests.Response = ac_client.get(
+                "applicaties", params={"clientIds": client_id}
             )
-        except requests.ConnectionError:
+
+            response.raise_for_status()
+        except requests.RequestException:
             error = True
             message = _("Could not connect with AC")
-        except ClientError as exc:
-            error = True
-            message = _(
-                "Cannot retrieve authorizations: HTTP {status_code} - {error_code}"
-            ).format(status_code=exc.args[0]["status"], error_code=exc.args[0]["code"])
         else:
             message = _("Can retrieve authorizations")
 
@@ -166,14 +175,13 @@ def _test_ac_config() -> list:
 
 
 def _test_nrc_config() -> list:
-    if not apps.is_installed("vng_api_common.notifications"):
+    if not apps.is_installed("notifications_api_common"):
         return []
 
     from notifications_api_common.models import NotificationsConfig, Subscription
 
     nrc_config = NotificationsConfig.get_solo()
-
-    nrc_client = NotificationsConfig.get_client()
+    nrc_client: Optional[Client] = NotificationsConfig.get_client()
 
     has_nrc_auth = nrc_client.auth is not None if nrc_client else False
 
@@ -199,15 +207,11 @@ def _test_nrc_config() -> list:
         error = False
 
         try:
-            nrc_client.list("kanaal")
-        except requests.ConnectionError:
+            response: requests.Response = nrc_client.get("kanaal")
+            response.raise_for_status()
+        except requests.RequestException:
             error = True
             message = _("Could not connect with NRC")
-        except ClientError as exc:
-            error = True
-            message = _(
-                "Cannot retrieve kanalen: HTTP {status_code} - {error_code}"
-            ).format(status_code=exc.args[0]["status"], error_code=exc.args[0]["code"])
         else:
             message = _("Can retrieve kanalen")
 

vng_api_common/notifications/constants.py

@@ -1,3 +0,0 @@
-# Exernally defined scopes.
-SCOPE_NOTIFICATIES_CONSUMEREN_LABEL = "notificaties.consumeren"
-SCOPE_NOTIFICATIES_PUBLICEREN_LABEL = "notificaties.publiceren"

vng_api_common/notifications/models.py

@@ -1,97 +0,0 @@
-import uuid
-from urllib.parse import urljoin
-
-from django.contrib.postgres.fields import ArrayField
-from django.db import models
-from django.utils.translation import gettext_lazy as _
-
-from zds_client import ClientAuth
-
-from ..client import get_client
-from ..decorators import field_default
-from ..models import APICredential, ClientConfig
-
-
-@field_default("api_root", "https://notificaties-api.vng.cloud/api/v1/")
-class NotificationsConfig(ClientConfig):
-    class Meta:
-        verbose_name = _("Notificatiescomponentconfiguratie")
-
-    def get_auth(self) -> ClientAuth:
-        auth = APICredential.get_auth(self.api_root)
-        return auth
-
-
-class Subscription(models.Model):
-    """
-    A single subscription.
-
-    TODO: on change/update, update the subscription
-    """
-
-    config = models.ForeignKey("NotificationsConfig", on_delete=models.CASCADE)
-
-    callback_url = models.URLField(
-        _("callback url"), help_text=_("Where to send the notifications (webhook url)")
-    )
-    client_id = models.CharField(
-        _("client ID"),
-        max_length=50,
-        help_text=_("Client ID to construct the auth token"),
-    )
-    secret = models.CharField(
-        _("client secret"),
-        max_length=50,
-        help_text=_("Secret to construct the auth token"),
-    )
-    channels = ArrayField(
-        models.CharField(max_length=100),
-        verbose_name=_("channels"),
-        help_text=_("Comma-separated list of channels to subscribe to"),
-    )
-
-    _subscription = models.URLField(
-        _("NC subscription"),
-        blank=True,
-        editable=False,
-        help_text=_("Subscription as it is known in the NC"),
-    )
-
-    class Meta:
-        verbose_name = _("Webhook subscription")
-        verbose_name_plural = _("Webhook subscriptions")
-
-    def __str__(self):
-        return f"{', '.join(self.channels)} - {self.callback_url}"
-
-    def register(self) -> None:
-        """
-        Registers the webhook with the notification component.
-        """
-        dummy_detail_url = urljoin(self.config.api_root, f"foo/{uuid.uuid4()}")
-        client = get_client(dummy_detail_url)
-
-        # This authentication is for the NC to call us. Thus, it's *not* for
-        # calling the NC to create a subscription.
-        self_auth = ClientAuth(
-            client_id=self.client_id,
-            secret=self.secret,
-        )
-        data = {
-            "callbackUrl": self.callback_url,
-            "auth": self_auth.credentials()["Authorization"],
-            "kanalen": [
-                {
-                    "naam": channel,
-                    # FIXME: You need to be able to configure these.
-                    "filters": {},
-                }
-                for channel in self.channels
-            ],
-        }
-
-        # register the subscriber
-        subscriber = client.create("abonnement", data=data)
-
-        self._subscription = subscriber["url"]
-        self.save(update_fields=["_subscription"])