Skip to content

Conversation

CathyDingemanse
Copy link
Collaborator

Ik heb geprobeerd de rules te versimpelen met het oog op het algoritmeregister.

Ik heb geprobeerd de rules te versimpelen met het oog op het algoritmeregister.
@CathyDingemanse CathyDingemanse requested a review from MelvLee June 5, 2024 08:31
Wanneer de afnemer een gemeente is, is er in de claims in het OAuth token ook een gemeentecode opgenomen. Deze wordt gebruikt om te bepalen of het bevraagde adres binnen de eigen gemeente ligt.
Autorisatie voor bewoning wordt bepaald door de gemeente waar het gevraagde adresseerbaar object ligt in de gevraagde periode.
Een afnemer is alleen geautoriseerd voor een vraag naar bewoning, wanneer in het antwoord op die vraag elke (mogelijke) bewoner in de periode van betreffende bewoning in de afnemer gemeente stond ingeschreven.
Wanneer de afnemer een gemeente is, bevat het OAuth token een gemeentecode. Deze wordt gebruikt om te bepalen of het bevraagde adres binnen de eigen gemeente ligt.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

gemeentecode => gemeentecode claim



Regel: Een gemeente als afnemer is geautoriseerd voor het vragen van bewoning waarbij voor de gevonden verblijfplaats(en) de gemeente van inschrijving gelijk is aan de gemeentecode in de 'claim' uit de token
Regel: Een gemeente als afnemer is geautoriseerd voor het vragen van bewoning waarbij de gemeente van de gevonden verblijfplaats(en) gelijk is aan de gemeentecode van de afnemer.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Dit is volgens mij niet wat wij afgesproken hebben. In dit geval moeten de verblijfplaatsen zijn opgehaald om dit te bepalen. We zouden het nu bepalen adhv de gemeentecode van de adres voorkomens van het adresseerbaar object. Volgens mij zou deze regel nu moeten zijn:

Een gemeente als afnemer is geautoriseerd voor het vragen van de bewoning van een adresseerbaar object als het adresseerbaar object ooit in de gemeente van de afnemer heeft gelegen

Een afnemer is alleen geautoriseerd voor een vraag naar bewoning, wanneer in het antwoord op die vraag elke (mogelijke) bewoner in de periode van betreffende bewoning in de afnemer gemeente stond ingeschreven.
Wanneer de afnemer een gemeente is, bevat het OAuth token een gemeentecode. Deze wordt gebruikt om te bepalen of het bevraagde adres binnen de eigen gemeente ligt.
Autorisatie voor bewoning wordt bepaald door de gemeente waar het gevraagde adresseerbaar object ligt.
Een afnemer is alleen geautoriseerd wanneer iedere (mogelijke) bewoner in de gevraagde periode van bewoning in de afnemergemeente was ingeschreven.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Deze regel kan volgens mij weg

| instance | /haalcentraal/api/bewoning/bewoningen |

Scenario: Gemeente raadpleegt bewoning van een adresseerbaar object binnen de gemeente en de bewoner is nu niet meer ingeschreven in de gemeente
Scenario: Gemeente raadpleegt bewoning van een adresseerbaar object binnen de gemeente waarvan de bewoner nu niet meer is ingeschreven in de gemeente
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Deze scenario doet niet meer toe. Er wordt alleen gekeken of het adresseerbaar object ooit in de gemeente van de afnemer heeft gelegen

Copy link
Contributor

@MelvLee MelvLee left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Volgens mij voldoen er veel scenarios niet. In de scenarios wordt nog steeds gesproken over verblijfplaatsen van personen. Volgens mij moet er alleen worden gekeken of één van de gemeentecode van de adres voorkomens van het adresseerbaar object overeenkomen met de gemeentecode claim van de afnemer

fsamwel added a commit that referenced this pull request Jun 10, 2024
Base automatically changed from autorisatie-bewoning to main October 29, 2024 09:55
Copy link

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants