Altinn / Altinn/altinn-auth

Delegation policy får ikke samme match algoritme som original ressurs/app policy

Open
#321 0 comments 0 reactions 0 assignees View on GitHub
Backend kind/bug
Dominant language
C#
Stars
4
Forks
3
Avg merge
2d 21h
Merged PRs (30d)
59

Description

### Description of the bug

Match algoritme er nå hardkodet ved opprettelse av delegation policy, i stedet for å hente dette fra original regel fra ressurs/app policy.

Dette var en forenkling som ble gjort ved første implementasjon av policy bygger for app-delegeringer, som siden ikke har blitt gjort oppdateringer på.

### Steps To Reproduce

1. Opprett en test ressurs i Ressursregisteret
2. I Policy for ressursen benytt case insensitiv match algoritme på f.eks Action:
```

read

```
3. PDP Decision/Authorize sjekk støtter da `case-insensitiv` sjekk for de med tilgang gjennom rolle/tilgangspakke, for Action: `Read`
4. Deleger tilgang til ressursen til noen andre
5. Regler i opprettet delegation policy er nå da hardkodet til å benytte case-sensitiv match algoritme:
```

read

```
6. PDP Decision/Authorize sjekk er da `case-sensitiv` for de med tilgang gjennom delegering, og vil feile dersom man sjekker for: `Read`

### Additional Information

Rapportert fra tjenesteeier i public slack:
https://altinn.slack.com/archives/C068V9FJQTD/p1738842166036469

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by tracing delegation policy creation and compare its generated match algorithm with the original resource/app policy. Reproduce the case-insensitive Action scenario from the issue and inspect the resulting XACML rules. Done means delegated authorization preserves the original MatchId and accepts the same Action values.

Written by the indexing model from the issue text.

Assessment

Tech stack
csharp
Domain
authorization, backend
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.