Move security files and remove roles
This MR attacks the scatteredness and complicatedness of our Security configurations.
I would like to propose we move all security related classes into a package 'security' for the sake of having this bubble of connected classes in the same place. This helps in understanding which classes do what in the security hierarchy, rather than having AuthenticationService (which is a UserDetailsService) in the services directory.
Merge request reports
Activity
- Resolved by Chris Lemaire
added 1 commit
- b26dd0a3 - Apply suggestion to src/main/java/nl/tudelft/ewi/queue/security/dev/AuthenticationService.java
added 56 commits
-
b26dd0a3...5dc9cb1a - 54 commits from branch
development
- acb2fcd1 - Move security files and remove roles
- 8d47ffba - Apply suggestion to src/main/java/nl/tudelft/ewi/queue/security/dev/AuthenticationService.java
-
b26dd0a3...5dc9cb1a - 54 commits from branch
- Resolved by Chris Lemaire
@Lemaire Is it possible that the current tests are failing because of the
@Profile('!development')
??
added 1 commit
- 310274f4 - Apply suggestion to src/main/java/nl/tudelft/ewi/queue/security/dev/AuthenticationService.java
- Resolved by Otto Visser
Shouldn't the SAML files then have profile production? As the dev files are for all non-production profiles.
added 274 commits
-
310274f4...69e5f575 - 272 commits from branch
development
- 0c3be689 - Move security files and remove roles
- d38da8e2 - Apply suggestion to src/main/java/nl/tudelft/ewi/queue/security/dev/AuthenticationService.java
-
310274f4...69e5f575 - 272 commits from branch
added 1 commit
- 6f274abc - Apply suggestion to src/main/java/nl/tudelft/ewi/queue/security/dev/AuthenticationService.java
added 1 commit
- 1deab638 - Apply suggestion to src/main/java/nl/tudelft/ewi/queue/security/dev/AuthenticationService.java
mentioned in commit a8110b99
mentioned in commit 7290a683