Skip to content

Commit

Permalink
👌 [#1471] PR feedback
Browse files Browse the repository at this point in the history
  • Loading branch information
SilviaAmAm committed Apr 19, 2022
1 parent 835676e commit 718ef9a
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 11 deletions.
12 changes: 5 additions & 7 deletions docs/configuration/authentication/oidc_digid_machtigen.rst
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand All @@ -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:
Expand All @@ -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 <configuration_oidc_digid_machtigen_appgroup>` zijn nodig:
Om OIDC in Open-Formulieren te kunnen configureren zijn de volgende :ref:`gegevens <configuration_oidc_digid_machtigen_appgroup>` nodig:

* Server adres
* Client ID
Expand All @@ -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.
Expand Down
5 changes: 1 addition & 4 deletions src/digid_eherkenning_oidc_generics/mixins.py
Original file line number Diff line number Diff line change
@@ -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,
Expand Down

0 comments on commit 718ef9a

Please sign in to comment.