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

Authentication by OAuth2 flow #948

Draft
wants to merge 36 commits into
base: main
Choose a base branch
from

Conversation

SebastianRossa
Copy link
Contributor

RISDEV-6410

SebastianRossa and others added 30 commits January 24, 2025 15:36
- remove session based authentication
- add redirect url in local keycloak for frontend to ease local development
- remove redis config

RISDEV-6410
That way we keep our main.ts a bit more tidy.

RISDEV-0000
Somehow lefthook messed this file up when some validation failed :(

RISDEV-6420
Fixes an issue where default values are required in tests, and is also more consistent
with how variables are handled in other places in the app.

RISDEV-6423, RISDEV-5914
Since this endpoint does not do anything related to our domain or services, the
related architecture tests are not applicable to it.

RISDEV-6423, RISDEV-5914
Fixes a SonarCloud complaint.

RISDEV-6423, RISDEV-5914
RISDEV-0000
This changes how the token is managed and how authorization headers are added
in tests such that it requires less repetitive boilerplate.

RISDEV-6410
It still fails because logging out is not implemented yet.

RISDEV-6410
- Imported `useAuthentication` and `addAuthorizationHeader` to attach auth headers
- Implemented `addAuthHeader` function to set headers before sending requests
- Used `@before-send="addAuthHeader"` to ensure authenticated requests

This fixes missing authorization headers when uploading files.

RISDEV-6410
andreasphil and others added 6 commits February 3, 2025 07:57
- Added error message in `index.html` inside `.fallback`
- Wrapped app initialization in a `try/catch` to handle failures in environment fetching and `auth.configure`
- If an error, loading message is hidden and default error appears with the error message (if exists)

RISDEV-6426
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants