From 718ef9aa1a3f54c4ec80b39ad15a69a0344ed2b4 Mon Sep 17 00:00:00 2001 From: SilviaAmAm Date: Tue, 19 Apr 2022 16:30:52 +0200 Subject: [PATCH] :ok_hand: [#1471] PR feedback --- .../authentication/oidc_digid_machtigen.rst | 12 +++++------- src/digid_eherkenning_oidc_generics/mixins.py | 5 +---- 2 files changed, 6 insertions(+), 11 deletions(-) diff --git a/docs/configuration/authentication/oidc_digid_machtigen.rst b/docs/configuration/authentication/oidc_digid_machtigen.rst index c5990602c7..6b31f33356 100644 --- a/docs/configuration/authentication/oidc_digid_machtigen.rst +++ b/docs/configuration/authentication/oidc_digid_machtigen.rst @@ -8,7 +8,7 @@ Open Formulieren ondersteunt `DigiD Machtigen`_ login voor burgers via het OpenI Burgers kunnen inloggen op Open Formulieren met hun DigiD account en een formulier invullen namens iemand anders. In deze flow: -* Een gebruiker klikt op de knop *Inloggen met DigiD Machtigen* die op de start pagina van een formulier staat. +* Een gebruiker klikt op de knop *Inloggen met DigiD Machtigen* die op de startpagina van een formulier staat. * De gebruiker wordt via de omgeving van de OpenID Connect provider (bijv. `Keycloak`_) naar DigiD geleid, waar de gebruiker kan inloggen met *zijn/haar eigen* DigiD inlog gegevens. * De gebruiker kan dan kiezen namens wie hij/zij het formulier wilt invullen. * DigiD stuurt de gebruiker terug naar de OIDC omgeving, die op zijn beurt de gebruiker weer terugstuurt naar Open Formulieren @@ -23,7 +23,7 @@ Configureren van OIDC voor DigiD Machtigen ========================================== De stappen hier zijn dezelfde als voor :ref:`configuration_oidc_digid_appgroup`, maar de **Redirect URI** -is ``https://open-formulieren.gemeente.nl/digid-oidc-machtigen/callback/`` (met de juiste domein in plaats van +is ``https://open-formulieren.gemeente.nl/digid-oidc-machtigen/callback/`` (met het juiste domein in plaats van ``open-formulieren.gemeente.nl``). Aan het eind van dit proces moet u de volgende gegevens hebben: @@ -35,7 +35,7 @@ Aan het eind van dit proces moet u de volgende gegevens hebben: Configureren van OIDC in Open Formulieren ========================================= -Om OIDC in Open-Formulieren te kunnen configureren, de volgende :ref:`gegevens ` zijn nodig: +Om OIDC in Open-Formulieren te kunnen configureren zijn de volgende :ref:`gegevens ` nodig: * Server adres * Client ID @@ -49,10 +49,8 @@ Navigeer vervolgens in de admin naar **Configuratie** > **OpenID Connect configu #. Vul bij **OpenID Connect scopes** ``openid``. #. Vul bij **OpenID sign algorithm** ``RS256`` in. #. Laat **Sign key** leeg. -#. Laat bij **Vertegenwoordigde claim name** de standaardwaarde staan, tenzij de naam van het BSN veld van de vertegenwoordigde -#. in de OIDC claims anders is dan ``aanvrager.bsn``. -#. Laat bij **Gemachtigde claim name** de standaardwaarde staan, tenzij de naam van het BSN veld van de gemachtigde -#. in de OIDC claims anders is dan ``gemachtigde.bsn``. +#. Laat bij **Vertegenwoordigde claim name** de standaardwaarde staan, tenzij de naam van het BSN veld van de vertegenwoordigde in de OIDC claims anders is dan ``aanvrager.bsn``. +#. Laat bij **Gemachtigde claim name** de standaardwaarde staan, tenzij de naam van het BSN veld van de gemachtigde in de OIDC claims anders is dan ``gemachtigde.bsn``. De endpoints die ingesteld moeten worden zijn dezelfde als voor DigiD. U kunt de stappen in :ref:`configuration_oidc_digid_appgroup` volgen om die te configureren. diff --git a/src/digid_eherkenning_oidc_generics/mixins.py b/src/digid_eherkenning_oidc_generics/mixins.py index 9cb0de09af..378f85dd2b 100644 --- a/src/digid_eherkenning_oidc_generics/mixins.py +++ b/src/digid_eherkenning_oidc_generics/mixins.py @@ -1,9 +1,6 @@ from mozilla_django_oidc_db.mixins import SoloConfigMixin as _SoloConfigMixin -import digid_eherkenning_oidc_generics.digid_machtigen_settings as digid_machtigen_settings -import digid_eherkenning_oidc_generics.digid_settings as digid_settings -import digid_eherkenning_oidc_generics.eherkenning_settings as eherkenning_settings - +from . import digid_machtigen_settings, digid_settings, eherkenning_settings from .models import ( OpenIDConnectDigiDMachtigenConfig, OpenIDConnectEHerkenningConfig,