feat: add authentik for authentication #58

Merged
jank merged 23 commits from feature/authentik into main 2025-04-04 13:26:03 +00:00
Owner
No description provided.
csimonis added 3 commits 2025-03-12 13:46:20 +00:00
csimonis force-pushed feature/authentik from 206eee4085 to 8939c7be79 2025-03-12 13:46:30 +00:00 Compare
csimonis force-pushed feature/authentik from 834ed7bbfc to 242b72ca45 2025-03-26 10:12:20 +00:00 Compare
csimonis added 1 commit 2025-03-26 12:27:43 +00:00
feat(security): add CORS support and update security config
Some checks failed
CI / Get Changed Files (pull_request) Successful in 9s
CI / eslint (pull_request) Failing after 29s
CI / prettier (pull_request) Failing after 32s
CI / test-build (pull_request) Failing after 58s
CI / Checkstyle Main (pull_request) Successful in 1m24s
3da534f3ae
jank added the
Kind/Enhancement
Priority
Medium
labels 2025-03-26 12:31:41 +00:00
csimonis added 1 commit 2025-04-02 11:07:19 +00:00
feat(login): log user info on successful login success page
Some checks failed
CI / Get Changed Files (pull_request) Successful in 8s
CI / eslint (pull_request) Failing after 24s
CI / test-build (pull_request) Failing after 30s
CI / prettier (pull_request) Failing after 50s
CI / Checkstyle Main (pull_request) Successful in 1m19s
7eebd12699
jank added 2 commits 2025-04-02 13:52:20 +00:00
refactor: rename keycloakId to authentikId in codebase
Some checks failed
CI / Get Changed Files (pull_request) Successful in 6s
CI / prettier (pull_request) Failing after 46s
CI / Checkstyle Main (pull_request) Successful in 49s
CI / eslint (pull_request) Failing after 1m2s
CI / test-build (pull_request) Failing after 1m9s
8317349507
Merge pull request 'refactor: rename keycloakId to authentikId in codebase' (!122) from fix-authentik into feature/authentik
Some checks failed
CI / Get Changed Files (pull_request) Successful in 6s
CI / eslint (pull_request) Failing after 20s
CI / test-build (pull_request) Failing after 24s
CI / prettier (pull_request) Failing after 51s
CI / Checkstyle Main (pull_request) Successful in 2m11s
80d5c1e413
Reviewed-on: #122
jank added 1 commit 2025-04-02 14:02:20 +00:00
Merge branch 'main' into feature/authentik
Some checks failed
CI / Get Changed Files (pull_request) Successful in 6s
CI / prettier (pull_request) Failing after 22s
CI / Checkstyle Main (pull_request) Failing after 35s
CI / eslint (pull_request) Failing after 1m41s
CI / test-build (pull_request) Successful in 1m48s
d7fe0e3965
jank added 1 commit 2025-04-02 14:09:39 +00:00
refactor(deposit, user): rename Keycloak to Authentik user info
Some checks failed
CI / Get Changed Files (pull_request) Successful in 6s
CI / eslint (pull_request) Failing after 25s
CI / test-build (pull_request) Successful in 33s
CI / prettier (pull_request) Failing after 56s
CI / Checkstyle Main (pull_request) Successful in 2m0s
fa09a8533f
jank added 1 commit 2025-04-02 14:11:58 +00:00
style: Fix formatting and spacing in multiple files
Some checks failed
CI / Get Changed Files (pull_request) Successful in 7s
CI / prettier (pull_request) Successful in 22s
CI / Checkstyle Main (pull_request) Successful in 53s
CI / eslint (pull_request) Failing after 1m8s
CI / test-build (pull_request) Successful in 1m35s
617654caeb
jank added 1 commit 2025-04-02 14:15:41 +00:00
refactor: improve type annotations in services and config
Some checks failed
CI / Get Changed Files (pull_request) Successful in 6s
CI / prettier (pull_request) Successful in 23s
CI / Checkstyle Main (pull_request) Successful in 31s
CI / eslint (pull_request) Successful in 1m30s
CI / test-build (pull_request) Failing after 1m43s
d3b7e7d5e7
jank added 1 commit 2025-04-02 14:20:42 +00:00
refactor: update imports and type definitions in services
Some checks failed
CI / Get Changed Files (pull_request) Successful in 7s
CI / eslint (pull_request) Successful in 26s
CI / test-build (pull_request) Successful in 31s
CI / prettier (pull_request) Failing after 54s
CI / Checkstyle Main (pull_request) Successful in 1m57s
e37dcecd3f
jank added 1 commit 2025-04-02 14:22:03 +00:00
style(user.service.ts): format code for clarity and consistency
All checks were successful
CI / Get Changed Files (pull_request) Successful in 7s
CI / prettier (pull_request) Successful in 26s
CI / Checkstyle Main (pull_request) Successful in 41s
CI / test-build (pull_request) Successful in 41s
CI / eslint (pull_request) Successful in 51s
47f4a4d558
jank added 1 commit 2025-04-02 14:24:45 +00:00
feat(auth): improve logout functionality and token management
Some checks failed
CI / Get Changed Files (pull_request) Successful in 6s
CI / eslint (pull_request) Successful in 26s
CI / test-build (pull_request) Successful in 33s
CI / prettier (pull_request) Failing after 50s
CI / Checkstyle Main (pull_request) Successful in 1m7s
2e76446328
jank added 1 commit 2025-04-02 14:27:40 +00:00
refactor: remove debug logs from auth components
Some checks failed
CI / Get Changed Files (pull_request) Successful in 7s
CI / prettier (pull_request) Failing after 23s
CI / eslint (pull_request) Failing after 28s
CI / test-build (pull_request) Successful in 1m16s
CI / Checkstyle Main (pull_request) Successful in 1m22s
9de08ab233
jank added 1 commit 2025-04-02 14:34:11 +00:00
refactor(auth): clean up login and logout logic
Some checks failed
CI / Get Changed Files (pull_request) Successful in 7s
CI / eslint (pull_request) Successful in 25s
CI / test-build (pull_request) Successful in 32s
CI / prettier (pull_request) Failing after 59s
CI / Checkstyle Main (pull_request) Successful in 1m29s
0e1946d190
Owner

thank me laterimage

thank me later![image](/attachments/0d3eef9d-f622-47e6-bebb-a7510efd5e57)
jank added 1 commit 2025-04-02 14:36:30 +00:00
refactor(login-success): remove unnecessary blank line
All checks were successful
CI / Get Changed Files (pull_request) Successful in 6s
CI / prettier (pull_request) Successful in 23s
CI / Checkstyle Main (pull_request) Successful in 31s
CI / eslint (pull_request) Successful in 1m20s
CI / test-build (pull_request) Successful in 1m37s
64f701c651
jank changed title from WIP: feat: add authentik for authentication to feat: add authentik for authentication 2025-04-02 14:40:46 +00:00
ptran requested changes 2025-04-03 07:22:47 +00:00
Dismissed
@ -20,3 +20,3 @@
private Long id;
@Column(unique = true)
private String keycloakId;
private String authentikId; // Changed from keycloakId to authentikId

Cool info, but not needed

Cool info, but not needed
jank marked this conversation as resolved
@ -67,0 +70,4 @@
return response.getBody();
} catch (Exception e) {
System.err.println("Error fetching user info from Authentik: " + e.getMessage());

Consider throwing an exception idk

Consider throwing an exception idk

why tho what is that gonna do?

why tho what is that gonna do?

nvm at that point I could also just remove the catch here

nvm at that point I could also just remove the catch here
jank marked this conversation as resolved
@ -10,6 +10,8 @@ import lombok.Setter;
@AllArgsConstructor
@NoArgsConstructor
public class KeycloakUserDto {
// Renamed class but kept for backward compatibility

ok? Can be removed or renamed if needed

ok? Can be removed or renamed if needed
jank marked this conversation as resolved
@ -11,2 +11,4 @@
@NoArgsConstructor
public class KeycloakUserDto {
// Renamed class but kept for backward compatibility
// This now contains Authentik user info

cool

cool
jank marked this conversation as resolved
@ -22,3 +20,1 @@
sessionStorage.setItem('user', JSON.stringify(user));
this.router.navigate(['home']);
try {

Remove comments

Remove comments
jank marked this conversation as resolved
@ -0,0 +45,4 @@
this.oauthService.configure(this.authConfig);
this.setupEventHandling();
// Check if we're on the callback page

useless

useless
jank marked this conversation as resolved
@ -55,2 +46,2 @@
return user;
}
public getOrCreateUser(profile: Record<string, unknown>): Observable<User> {
// Authentik format might differ from Keycloak

cool bro

cool bro
jank marked this conversation as resolved
jank added 1 commit 2025-04-03 09:45:04 +00:00
Merge branch 'main' into feature/authentik
All checks were successful
CI / Get Changed Files (pull_request) Successful in 6s
CI / prettier (pull_request) Successful in 55s
CI / Checkstyle Main (pull_request) Successful in 1m1s
CI / eslint (pull_request) Successful in 1m43s
CI / test-build (pull_request) Successful in 1m55s
25c68e230d
jank added 1 commit 2025-04-03 09:52:05 +00:00
refactor(user): clean up comments and rename variables
All checks were successful
CI / Get Changed Files (pull_request) Successful in 6s
CI / eslint (pull_request) Successful in 25s
CI / test-build (pull_request) Successful in 31s
CI / prettier (pull_request) Successful in 56s
CI / Checkstyle Main (pull_request) Successful in 2m12s
02453449cd
ptran requested changes 2025-04-03 09:56:22 +00:00
Dismissed
@ -30,3 +11,1 @@
redirectUri: window.location.origin + '/',
},
});
/* Example of a custom storage factory - not used in the current implementation

/* Example of a custom storage factory - not used in the current implementation
???

/* Example of a custom storage factory - not used in the current implementation ???
jank marked this conversation as resolved
ptran reviewed 2025-04-03 09:56:57 +00:00
@ -1,6 +1,7 @@
import { ChangeDetectionStrategy, Component, inject, signal, OnInit } from '@angular/core';
import { CommonModule } from '@angular/common';
import { Router } from '@angular/router';
// Importing components used in templates

Remove comment

Remove comment
jank marked this conversation as resolved
jank added 1 commit 2025-04-03 09:59:10 +00:00
refactor: remove unused storage factory code and comments
Some checks failed
CI / Get Changed Files (pull_request) Successful in 6s
CI / eslint (pull_request) Successful in 24s
CI / test-build (pull_request) Successful in 31s
CI / prettier (pull_request) Failing after 56s
CI / Checkstyle Main (pull_request) Successful in 1m30s
c765ef87e3
jank added 1 commit 2025-04-03 10:03:07 +00:00
style: remove unnecessary blank line in config file
Some checks failed
CI / Get Changed Files (pull_request) Successful in 5s
CI / Checkstyle Main (pull_request) Has been cancelled
CI / eslint (pull_request) Has been cancelled
CI / prettier (pull_request) Has been cancelled
CI / test-build (pull_request) Has been cancelled
c942b4bb39
requested review from ptran 2025-04-03 14:21:13 +00:00
jank added this to the Sprint 2 milestone 2025-04-04 11:02:41 +00:00
jank scheduled this pull request to auto merge when all checks succeed 2025-04-04 12:20:56 +00:00
ptran approved these changes 2025-04-04 12:20:58 +00:00
jank merged commit b803055307 into main 2025-04-04 13:26:03 +00:00
jank deleted branch feature/authentik 2025-04-04 13:26:03 +00:00
csimonis deleted branch feature/authentik 2025-04-25 06:01:58 +00:00
Commenting is not possible because the repository is archived.
No reviewers
No milestone
No project
No assignees
3 participants
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: SZUT/casino#58
No description provided.