-
Notifications
You must be signed in to change notification settings - Fork 4
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
EndUser Transmission endpoints does not mask unauthorized attachment URLs #1889
Comments
…n endpoints (#1890) <!--- Provide a general summary of your changes in the Title above --> ## Description EndUser transmission endpoints gives out original URLs when the user is not authorized to see them ## Related Issue(s) - #1889 ## Verification - [x] **Your** code builds clean without any errors or warnings - [x] Manual testing done (required) - [ ] 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)
Test🟢 Har testet ut fra beskrivelsen i saken og det fungerer. URL-ene til vedleggene blir maskert. Basert på dokumentasjonen https://docs.altinn.studio/dialogporten/reference/entities/transmission/ burde man for authorizationAttribute også kunne angitt en URN. Men her kan ikke avsender angi hva som helst, det er en URN-validering. 🟢 Test med en URN som org kan sende for, men som party her ikke kan åpne (muligens litt tilfeldig) |
Description
EndUser transmission endpoints gives out original URLs when the user is not authorized to see them
Reproduction
Create a transmission containing attachments, and with an authorization attribute you know the user does not have,
not-authorized
f.ex.Fetch the dialog at
api/v1/enduser/dialogs/{dialogId}
, attachment URLs are all swapped out with our unauthorized URI.Get the transmission via either single or list endpoint,
/api/v1/enduser/dialogs/{dialogId}/transmissions/{transmissionId}
and
/api/v1/enduser/dialogs/{dialogId}/transmissions
Expected behavior
URLs should be masked as
urn:dialogporten:unauthorized
Actual behavior
Original URL is shown in both endpoints
The text was updated successfully, but these errors were encountered: