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

Maskinporten auth #210

Merged
merged 26 commits into from
Dec 4, 2023
Merged

Maskinporten auth #210

merged 26 commits into from
Dec 4, 2023

Conversation

Ceredron
Copy link
Collaborator

@Ceredron Ceredron commented Dec 4, 2023 •
edited
Loading

Description

Added support for using Maskinporten token for the API.

Related Issue(s)

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 added 14 commits December 1, 2023 15:36
@Ceredron
Added code for parsing maskinporten token
86a3a37
@Ceredron
Updated controller logic naively to accomodate new authentication.
5167174
@Ceredron
Fix build
316b7d6
@Ceredron
Deleted dead code
ec633a6
@Ceredron
Some renaming
e22dbaf
@Ceredron
This value is neither dependent on environment nor something that nee…
2e63ac7 
…ds to be configured. Changed it to be a constant.
@Ceredron
Have to delete it too
4d3322d
@Ceredron
Fixed comment
9acde93
@Ceredron
Handle scope using attribute
5e39449
@Ceredron
Added Maskinporten authentication.
d7e3208
@Ceredron
Formatting
321f5ba
@Ceredron
Fixed issue where multiple scopes were used. Maskinporten sends token…
e9f7feb 
… in format not expected by asp net (space separated scopes and not array of strings). Postpone developing a convoluted solution to deal with this until authz figured out.
@Ceredron
File no longer in use
18dc023
@Ceredron
Found better way to configure JWT
dcf8b64
@Ceredron Ceredron changed the title Feature/maskinporten autha Maskinporten authentication Dec 4, 2023
@Ceredron Ceredron changed the title Maskinporten authentication Maskinporten auth Dec 4, 2023
@Ceredron Ceredron merged commit da41929 into main Dec 4, 2023
1 check passed
@Ceredron Ceredron deleted the Feature/maskinporten-autha branch December 4, 2023 15:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@Ceredron