Skip to content
New issue

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

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

Already on GitHub? # to your account

Always exchange to Altinn token when communicating with Altinn platform #167

Merged
merged 2 commits into from
Aug 1, 2024

Conversation

Ceredron
Copy link
Collaborator

@Ceredron Ceredron commented Aug 1, 2024

Description

We previously used Maskinporten token to Altinn Authorization, but that is no longer the case. Hence we move exhangeToAltinnToken to "global" configuration for our Altinn services.

There seems to have been an issue with cache invalidation where the authorization token, which was Maskinporten only, was not renewed as expected. Reading through the library, it does not seem like how we used the library (doing both Maskinporten and Altinn token at the same time) was the expected way to use it. Nor is it the way forward for us either.

Related Issue(s)

  • #507

Verification

  • Your code builds clean without any errors or warnings
  • Manual testing done (required)
  • Relevant automated test added (if you find this hard, leave it and we'll help out)
  • All tests run green

Documentation

  • User documentation is updated with a separate linked PR in altinn-studio-docs. (if applicable)

@Ceredron Ceredron linked an issue Aug 1, 2024 that may be closed by this pull request
@Ceredron Ceredron merged commit 77fef2c into main Aug 1, 2024
4 checks passed
@Ceredron Ceredron deleted the fix/always-exchange-to-altinn-token branch August 1, 2024 13:26
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Token ikke fornyet
2 participants