Extend authorized parties with instance delegations #856
Conversation
andreasisnes
force-pushed
the
issues/812
branch
from
25f21d6
to
35577ab
Compare
src/Altinn.AccessManagement.Core/Repositories/Interfaces/IDelegationMetadataRepository.cs
Outdated
Show resolved
Hide resolved
src/Altinn.AccessManagement.Core/Services/AuthorizedPartiesService.cs
Outdated
Show resolved
Hide resolved
src/Altinn.AccessManagement.Persistence/DelegationMetadataRepo.cs
Outdated
Show resolved
Hide resolved
| /// <summary> | ||
| /// Composite Key instances | ||
| /// </summary> | ||
| public class Resource |
There was a problem hiding this comment.
Burde ikke denne hete noe med Instance for ikke å blande den med AuthorizedResources som er av type string
There was a problem hiding this comment.
mm, skal rename den
There was a problem hiding this comment.
blir kanskje ikke så synelig i File Changes PRen. Men klassen er definert inne i klassen AuthorizedParty. Så for å referere til klassen Resource utenfor klassen AuthorizedParty så må man bruke AuthorizedParty.Resource. Uansett, så endret jeg navnet til AuthorizedResource. Så det blir nå AuthorizedParty.AuthorizedResource.
| @@ -164,10 +164,18 @@ public async Task<List<AuthorizedParty>> GetAuthorizedPartiesForEnterpriseUserUu | |||
| return await Task.FromResult(new List<AuthorizedParty>()); | |||
| } | |||
|
|
|||
| private async Task<List<InstanceDelegationChange>> GetInstanceDelegations(int subjectUserId, List<int> subjectPartyIds, CancellationToken cancellationToken) | |||
| { | |||
| var userId = subjectUserId != 0 ? subjectUserId.SingleToList() : []; | |||
There was a problem hiding this comment.
Det blir vel feil å legge inn en userid i en liste over partyids siden disse ikke brukes likt og ikke har samme verdi en user har en party id men den er ikke lik userid
There was a problem hiding this comment.
jaja, var en god del issues med denne logikken. Har skrevet den om nå, ble ganske feil.
|
|
||
| if (authorizedPartyDict.TryGetValue(instanceParty.PartyId, out var authorizedParty)) | ||
| { | ||
| continue; |
There was a problem hiding this comment.
Vil ikke denne continue blokken medføre at bare nye partyid får lagt til nye instanser om du har flere instanser så er det bare den første som blir lagt til og dersom du har en annen tilgang så blir ingen instanser lagt til siden den allerede ligger i dictionarien.
There was a problem hiding this comment.
mm, har skrevet om logikken her.
| WHERE | ||
| touuid = ANY(@toUuid) | ||
| GROUP BY | ||
| resourceid, touuid, fromuuid; |
There was a problem hiding this comment.
Samme som JK dersom du ikke har instanceId i group by så vil den slå sammen alle instansene desuten er det vel ikke mulig å bruke en group by og hente ut alle kolonner for de kolenne som ikke er grupert på vil kunne være mange verdier så de må ha en agregate funksjon for å finne hvilken som skal brukes. Jeg ville brukt Common table expression her og grupere på de dataene som var viktige og så hente ut max(instancedelegationchangeid) og deretter joine med denne common table expressionen med tabellen på nytt med id som match så ville jeg få ut alle siste endringer.
There was a problem hiding this comment.
stemmer, templating før jeg fikk testet spørringen. Endret på spørringen nu
test/Altinn.AccessManagement.Tests/Controllers/V2AuthorizedPartiesControllerTest.cs
Outdated
Show resolved
Hide resolved
|
Extend authorized parties with instance delegations.
Related Issue(s)
Developer/Reviewer Checklist
Documentation