Feature/no msal react #272
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Aqua-sc
approved these changes
May 20, 2024
There was a problem hiding this comment.
Heb een snel door de changed files geklikt maar lijkt me niet nuttig dat ik alles probeer te begrijpen wat er gebeurt. Op zicht zag het er prima uit (behalve de ene TODO)
Heb dan via frontend ook wat rondgeklikt en er was 1 klein moment dat ik een rare error kreeg maar heb het niet kunnen reproduceren dus ik denk dat dit wel fine is
backend/web-bff/App/routes/auth.js
Outdated
Comment on lines
21
to
32
| /** | ||
| * No longer used. TODO: remove | ||
| * | ||
| * Route that acquires a token for accessing the backend resource server. | ||
| * It stores this token in the session, it does not return the token. | ||
| * | ||
| * @route GET /web/auth/acquireToken | ||
| */ | ||
| router.get('/acquireToken', authProvider.acquireToken({ | ||
| scopes: [msalConfig.auth.clientId + "/.default"], | ||
| redirectUri: REDIRECT_URI, | ||
| successRedirect: '/users/profile' | ||
| redirectUri: REDIRECT_URI | ||
| })); |
There was a problem hiding this comment.
Als dit nog steeds niet meer gebruikt wordt mss best verwijderen. Als het wel nog gebruikt wordt comment aanpassen
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Express login server.
Deze pull request verwijderd al het gebruik van de MSAL react library in de frontend.
Het is nu vervangen door een auth context die de express server contacteert voor authenticatie status.
Inloggen wordt gedaan door te redirecten van de frontend naar de express server.
De express server behandelt dan de interactieve microsoft login en bewaart de ingelogde gebruiker in een mongodb database.
Er is een mapje temp-frontend. Deze is niet belangrijk en doet niks meer, maar is handig om te debuggen als het moet.
Hoe opzetten?
Backend
Zoals altijd, hier is niks veranderd.
Let op de application.properties en application-secrets.properties.
Express server
In de directory
backend/web-bff/App/voer jenpm run startom standaard te runnen ofnpm run devom te auto reloaden bij aanpassingen.Als je de mongodb storage wil testen, dan moet je zien dat er een mongodb community edition process op je computer aan het runnen is.
Ikzelf heb in de mongosh shell de database en user zelf aangemaakt voor te verbinden met de server.
Standaard wordt er in de lokale uitvoering geen mongodb storage gebruikt, maar gewoon in memory storage.
Ook hier is een environment file nodig. Voorzie een file .env in de directory
backend/web-bff/App/met de nodige environment variables. Zie discord channel testing-env.Frontend
Hier is niks veranderd.
npm run startuitvoeren in defrontend/directory.Problemen
Tot we de microsoft application kunnen laten redirecten naar
https://sel2-6.ugent.be/web, moeten we in productie een workaround doen met nginx, namelijk de POST van de/route naar de express server sturen, maar niet de GET. Voor deployen dus best eens overleggen.Voor productie moet ik ook een paar hard gecodeerde urls vervangen in de frontend.