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.

Sign up for GitHub

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

Jump to bottom

DataLocationUrl skal ikke være en verdi som brukere kan se #222

Closed
Andreass2 opened this issue Aug 23, 2024 · 0 comments · Fixed by #231
Closed

DataLocationUrl skal ikke være en verdi som brukere kan se #222

Andreass2 opened this issue Aug 23, 2024 · 0 comments · Fixed by #231
Assignees
@CelineTrammi
Labels
good-first-issue Issues that are easy to fix for new team members or external contributors kind/feature product/meldingstjenesten Issues related to Altinn Correspondence

Comments

@Andreass2
Copy link
Collaborator

Brukere laster ned vedlegg gjennom vårt API, derfor er det unødvendig å eksponere en ekstra angrepsflate ved å tilgjengeliggjøre DataLocationUrl til brukeren.

Fjern feltet fra alle responser det er inkludert i
@Andreass2 Andreass2 converted this from a draft issue Aug 23, 2024
@Andreass2 Andreass2 added good-first-issue Issues that are easy to fix for new team members or external contributors product/meldingstjenesten Issues related to Altinn Correspondence kind/feature labels Aug 23, 2024
@CelineTrammi CelineTrammi self-assigned this Aug 26, 2024
@CelineTrammi CelineTrammi moved this from Ready for dev to 🏗 In progress in Altinn melding og formidling Aug 26, 2024
@CelineTrammi CelineTrammi mentioned this issue Aug 27, 2024
Merged
5 tasks
@CelineTrammi CelineTrammi linked a pull request Aug 27, 2024 that will close this issue
Fix/remove data location url in responses #231
Merged
5 tasks
@CelineTrammi CelineTrammi moved this from 🏗 In progress to 👀 In review in Altinn melding og formidling Aug 29, 2024
@github-project-automation github-project-automation bot moved this from 👀 In review to ✅ Done in Altinn melding og formidling Aug 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@CelineTrammi CelineTrammi

Labels
good-first-issue Issues that are easy to fix for new team members or external contributors kind/feature product/meldingstjenesten Issues related to Altinn Correspondence
Projects
Altinn melding og formidling
Status: ✅ Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix/remove data location url in responses Altinn/altinn-correspondence
2 participants
@Andreass2 @CelineTrammi