Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Mottaker av flere enkeltdelegeringer fra samme avgiver kan bare bli autorisert for en av tjenestene innenfor cache tid av delegeringer (5 min) #1096

Closed
jonkjetiloye opened this issue Sep 27, 2024 · 1 comment
Assignees
Labels
kind/bug Used when something isn't working

Comments

@jonkjetiloye
Copy link
Member

Description of the bug

Cache nøkkel benyttet ved caching av respons fra PIP API i AccessManagement benytter ikke selve Org/App eller RessursId verdien fra PIP requesten. Dette betyr at to autorisasjonsforespørser for forskjellige Apps eller Ressurser som treffer samme pod innenfor cache tiden (5 minutter) så vil alltid request nr. 2 feile fordi den treffer cachet delegering fra den første requesten.

Omfang

Buggen har minst vært aktiv siden endring utført 06.05.24, men trolig også før dette så langt tilbake som 24.01.24.

Denne buggen treffer nok mest multirequests for Altinn app instanser i meldingsboksen, for brukere som har mottatt enkeltdelegering av mer enn en Altinn App, og avgiver har aktive instanser for mer enn en av appene som er delegert.

Da vil bare første (nyeste?) App instans bli autorisert mens påfølgende instanser ikke vil bli synlige i meldingsboksen.

Steps To Reproduce

Altinn App:

  1. Som Tilgangsstyrer for avgiver, deleger enkelttilgang for to Altinn Apps til en mottaker
  2. Som Tilgangsstyrer for avgiver, instansier begge appene så en instans av hver finnes i meldingsboksen
  3. Som mottaker av App delegeringene, logg på Altinn portal og naviger til meldingsboksen til avgiver

Forventet: At mottaker skal se begge instansene i meldingsboken og få tilgang til å åpne begge
Faktisk: Mottaker bare ser og får tilgang til en av instansene

Ressurs:

  1. Som Tilgangsstyrer for avgiver, deleger enkelttilgang for to Ressurser fra Ressursregisteret til mottaker
  2. Som Tjenesteeier, utfør en multirequest til Authorize API for begge ressursene delegert til mottakeren

Forventet: Decision Permitted for begge
Faktisk: Får Decision Not Applicable for Ressurs nr. 2 i multirequest.

Additional Information

No response

@jonkjetiloye jonkjetiloye added the kind/bug Used when something isn't working label Sep 27, 2024
@jonkjetiloye jonkjetiloye self-assigned this Sep 27, 2024
@jonkjetiloye jonkjetiloye moved this from New to 👷In Progress in Team Tilgangsstyring & Kontroll Sep 27, 2024
@jonkjetiloye jonkjetiloye changed the title Mottaker av flere enkeltdelegeringer kan bare bli autorisert for en tjeneste innenfor cache tid av delegeringer (5 min) Mottaker av flere enkeltdelegeringer fra samme avgiver kan bare bli autorisert for en av tjenestene innenfor cache tid av delegeringer (5 min) Sep 27, 2024
jonkjetiloye added a commit that referenced this issue Sep 30, 2024
* Make delegations PIP cachekey great again
#1096

* Added integration tests and Bruno automated tests
- Multi-request test for two Altinn Apps based on delegations
- Multi-request test for two Resourceregistry Resources based on delegations

---------

Co-authored-by: Jon Kjetil Øye <acn-joye@ai-dev.no>
@sneha-sirure
Copy link

@jonkjetiloye Det har jeg verifisert med positive og negative scenarioer og fungere som forventet nå i AT miljø

@github-project-automation github-project-automation bot moved this from 👷In Progress to ✅ Done in Team Tilgangsstyring & Kontroll Oct 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Used when something isn't working
Projects
Development

No branches or pull requests

2 participants