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

Discard the use of Altinn.Clients.Maskinporten package in Altinn.App.Core #706

Open
3 tasks
RonnyB71 opened this issue Jul 5, 2024 · 0 comments
Open
3 tasks

Comments

@RonnyB71
Copy link
Member

RonnyB71 commented Jul 5, 2024

Description

As part of the Maskinporten roadmap issue a new Maskinporten Client have been added to Altinn.App.Core. This effectively deprecates the use of the MaskinportenHttpClient from the Altinn.Clients.Maskinporten Nuget package.

With the exception of any custom maskinporten usage in applications we have three integrations using Maskinporten:

  1. eFormidling
  2. Altinn Event
  3. DAN

The new implementation have been tested and works "out-of-the-box" with DAN. However, our current eFormidling implementation, which in turn depends on Altinn Event, has a service that depends on IMaskinportenService and also a newer implementation depends on IMaskinportenTokenProvider.

We should remove the Altinn.Clients.Maskinporten Nuget package in favour of our own implementation and also rewrite existing integrations for eFormidling and Altinn Event.

Additional Information

No response

Tasks

No response

Acceptance Criterias

  • The Altinn.Clients.Maskinporten package should be removed
  • The new MaskinportenClient should replace existing Maskinporten based integrations
  • Migration script should be created for existing applications if possible
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: No status
Development

No branches or pull requests

1 participant