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

[9] Opprette AD-pålogging for Folk.knowit.no #577

Merged
merged 4 commits into from
May 28, 2024

Conversation

joacimds
Copy link
Collaborator

@joacimds joacimds commented May 21, 2024

Hva løser oppgaven:

Ønske om å gå vekk fra Google som identity provider og i stedet bruke Azure AD/Entra ID

Hvordan er oppgaven løst:

  • Byttet ut gammel autentiseringsflow gjennom OpenID-klient til Amplify. Wrappet appen i en <Authenticator.Provider>
  • Fjernet client_secret som nå er obsolete.
  • Tar i bruk imports fra Amplify, i.e. fetchAuthSession() og signInWithRedirect() til å håndtere innlogging, utlogging og access token.
  • Bruker nå LoginLogoutButton.tsx i headeren og i login siden, i stedet for at det er to ulike løsninger for dette.
  • Refaktorert koden relatert til UserInfoContext til å bli en context-fil og en hook.
  • Refaktorert ut HeaderTabs til en function component.
  • Generell opprydding i filer det er arbeidet på, som f. eks. imports.

@joacimds joacimds requested a review from TirilS May 21, 2024 11:52
@joacimds joacimds changed the title [9] Opprette AD-pålogging for Folk.knowit.no #9 [9] Opprette AD-pålogging for Folk.knowit.no May 21, 2024
@joacimds joacimds self-assigned this May 21, 2024
@joacimds joacimds added the enhancement New feature or request label May 21, 2024
@joacimds joacimds merged commit 6b55f7d into master May 28, 2024
3 checks passed
@joacimds joacimds deleted the feature/9-opprette-ad-palogging branch May 28, 2024 09:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants