Service resource references for apps use wrong format #681
Comments
elsand
moved this to 🔖 Klar for implementering
in ⚠️ Dialogporten / Arbeidsflate - GAMMEL - se https://github.com/orgs/Altinn/projects/146 ⚠️
elsand
moved this from 🔖 Klar for implementering
to Code Review og PR
in ⚠️ Dialogporten / Arbeidsflate - GAMMEL - se https://github.com/orgs/Altinn/projects/146 ⚠️
4 tasks
elsand
added a commit
that referenced
this issue
Apr 30, 2024
## Description This fixes the handling of service resource values referring to Altinn apps so that they too use the same prefix as generic resources. The decision request helper now only checks for the reserved resource name prefix (`app_`) in order to determine how the XACML request should be constructed. ## Related Issue(s) - #681 ## Verification - [x] **Your** code builds clean without any errors or warnings - [x] Manual testing done (required) - [x] Relevant automated test added (if you find this hard, leave it and we'll help out) ## Documentation - [ ] Documentation is updated (either in `docs`-directory, Altinnpedia or a separate linked PR in [altinn-studio-docs.](https://github.com/Altinn/altinn-studio-docs), if applicable)
elsand
moved this from Code Review og PR
to Testing / review
in ⚠️ Dialogporten / Arbeidsflate - GAMMEL - se https://github.com/orgs/Altinn/projects/146 ⚠️
4 tasks
github-project-automation
bot
moved this from Testing / Design QA
to Done
in ⚠️ Dialogporten / Arbeidsflate - GAMMEL - se https://github.com/orgs/Altinn/projects/146 ⚠️
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Description
See https://altinndevops.slack.com/archives/CCJJWCBJQ/p1714383798135549
Expected behavior
Apps must be referenced with
urn:altinn:resource:app_{app}_{org}Actual behavior
Apps must be referenced with
urn:altinn:app:app_{app}_{org}Additional information
app_is a reserved prefix, and all values can be assumed to have that format. This should trigger how XACML requests are built (ie. useurn:altinn:app/urn:altinn:orgas resource attribute ids instead ofurn:altinn:resource)The text was updated successfully, but these errors were encountered: