Skip to content

Team 10 ase 39 rbac for endpoints#9

Merged
davidclarafigueiredo merged 15 commits intomainfrom
team-10-ASE-39
Sep 22, 2025
Merged

Team 10 ase 39 rbac for endpoints#9
davidclarafigueiredo merged 15 commits intomainfrom
team-10-ASE-39

Conversation

@LucaSchmitz2003
Copy link
Copy Markdown
Contributor

No description provided.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds OAuth2 JWT authentication with Keycloak integration to the Spring Boot user service. The changes implement role-based access control using JWT tokens from a Keycloak realm.

  • Configures OAuth2 resource server with JWT authentication using Keycloak
  • Implements custom JWT authentication converter for role extraction and mapping
  • Adds role-based endpoint protection with a demo controller for testing

Reviewed Changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
src/main/resources/application.yaml Adds OAuth2 JWT configuration with Keycloak issuer and JWK set URIs, sets server port
src/main/java/com/ase/userservice/security/SecurityConfig.java Configures security filter chain with role-based authorization and JWT authentication
src/main/java/com/ase/userservice/security/JwtAuthConverter.java Custom converter to extract roles from JWT claims and map them to Spring Security authorities
src/main/java/com/ase/userservice/controllers/DemoController.java Simple demo controller with role-protected endpoint for testing authentication
pom.xml Adds Spring Security and OAuth2 resource server dependencies
Dockerfile Configures Docker container for running the Spring Boot application

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@ItsGamerik ItsGamerik changed the title Draft: Team 10 ase 39 Team 10 ase 39 rbac for endpoints Sep 22, 2025
@ItsGamerik ItsGamerik marked this pull request as ready for review September 22, 2025 11:00
ItsGamerik and others added 4 commits September 22, 2025 13:30
the formatter uses some kind of formatting i cant properly understand on my machine using the redhat java formatter.
@davidclarafigueiredo davidclarafigueiredo merged commit 32e3c67 into main Sep 22, 2025
0 of 2 checks passed
@LucaSchmitz2003 LucaSchmitz2003 deleted the team-10-ASE-39 branch September 22, 2025 14:08
davidclarafigueiredo pushed a commit that referenced this pull request Oct 17, 2025
* change deployment url

* fix deployment
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