-
Notifications
You must be signed in to change notification settings - Fork 55
Backlog/add oauth2 OIDC corporate authentication to backend #1413
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
base: v11
Are you sure you want to change the base?
Backlog/add oauth2 OIDC corporate authentication to backend #1413
Conversation
…or handling Signed-off-by: Manuel Abascal <mjabascal10@gmail.com>
…ser feedback Signed-off-by: Manuel Abascal <mjabascal10@gmail.com>
…n confirmation Signed-off-by: Manuel Abascal <mjabascal10@gmail.com>
…es, and expiration indicators Signed-off-by: Manuel Abascal <mjabascal10@gmail.com>
…ation Signed-off-by: Manuel Abascal <mjabascal10@gmail.com>
…tartQueue functions
…splay Signed-off-by: Manuel Abascal <mjabascal10@gmail.com>
…for syslog integrations Signed-off-by: Manuel Abascal <mjabascal10@gmail.com>
…-key-modal/api-key-modal.component.ts Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…yFilter.java Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
# Conflicts: # backend/src/main/java/com/park/utmstack/config/Constants.java
… and feedback Signed-off-by: Manuel Abascal <mjabascal10@gmail.com>
… utf8 characters even url structure ones
Signed-off-by: Manuel Abascal <mjabascal10@gmail.com>
Signed-off-by: Manuel Abascal <mjabascal10@gmail.com>
…e-authentication-to-backend' into backlog/add-oauth2-oidc-corporate-authentication-to-backend
… and event handling
…e-authentication-to-backend' into backlog/add-oauth2-oidc-corporate-authentication-to-backend
… and event handling
… and event handling
Signed-off-by: Manuel Abascal <mjabascal10@gmail.com>
…e-authentication-to-backend' into backlog/add-oauth2-oidc-corporate-authentication-to-backend
… and event handling
Signed-off-by: Manuel Abascal <mjabascal10@gmail.com>
…e-authentication-to-backend' into backlog/add-oauth2-oidc-corporate-authentication-to-backend
There was a problem hiding this 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 implements OAuth2/OIDC corporate authentication to the backend, enabling users to authenticate with external identity providers (Google, Microsoft) in addition to the existing username/password authentication.
Key changes:
- Adds backend support for OAuth2/OIDC authentication flows with dynamic provider configuration
- Creates a new frontend UI for identity provider management in admin settings
- Implements login provider selection on the login page
Reviewed Changes
Copilot reviewed 57 out of 57 changed files in this pull request and generated 10 comments.
Show a summary per file
| File | Description |
|---|---|
| frontend/src/environments/environment.ts | Switches server URL from production IP to localhost for development |
| frontend/src/app/shared/utm-shared.module.ts | Registers new LoginProvidersComponent in shared module |
| frontend/src/app/shared/services/login-provider.service.ts | Service to fetch active providers and initiate OAuth login flows |
| frontend/src/app/shared/components/auth/login/login.component.ts | Adds OAuth error handling and lifecycle management for login flow |
| frontend/src/app/shared/components/auth/login-providers/* | New component displaying OAuth provider buttons on login page |
| frontend/src/app/app-management/identity-provider/* | Complete provider management UI with CRUD operations |
| backend/src/main/resources/config/liquibase/changelog/20251029001-add-identity-provider-config.xml | Database schema for storing identity provider configurations |
| backend/src/main/java/com/park/utmstack/web/rest/idp_provider/* | REST endpoints for provider management and login provider listing |
| backend/src/main/java/com/park/utmstack/security/oauth/* | OAuth2 authentication handlers and custom user service |
| backend/src/main/java/com/park/utmstack/config/oauth/* | Dynamic OAuth2 client registration from database |
| backend/pom.xml | Adds spring-boot-starter-oauth2-client dependency |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
frontend/src/app/shared/components/auth/login/login.component.ts
Outdated
Show resolved
Hide resolved
frontend/src/app/shared/components/auth/login-providers/login-providers.component.ts
Outdated
Show resolved
Hide resolved
...nd/src/main/java/com/park/utmstack/web/rest/idp_provider/IdentityProviderConfigResource.java
Show resolved
Hide resolved
backend/src/main/java/com/park/utmstack/config/SecurityConfiguration.java
Outdated
Show resolved
Hide resolved
backend/src/main/java/com/park/utmstack/service/idp_provider/IdentityProviderService.java
Outdated
Show resolved
Hide resolved
backend/src/main/java/com/park/utmstack/config/SecurityConfiguration.java
Show resolved
Hide resolved
frontend/src/app/shared/components/auth/login-providers/login-providers.component.ts
Show resolved
Hide resolved
backend/src/main/java/com/park/utmstack/security/oauth/OAuth2LoginSuccessHandler.java
Show resolved
Hide resolved
backend/src/main/java/com/park/utmstack/domain/idp_provider/enums/ClientAuthMethod.java
Show resolved
Hide resolved
.../app-management/identity-provider/shared/components/provider-form/provider-form.component.ts
Outdated
Show resolved
Hide resolved
…display Signed-off-by: Manuel Abascal <mjabascal10@gmail.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…providers.component.ts Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…ration.java Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…dentityProviderService.java Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…nents/provider-form/provider-form.component.ts Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…uth2-oidc-corporate-authentication-to-backend
… and event handling
Signed-off-by: Manuel Abascal <mjabascal10@gmail.com>
…e-authentication-to-backend' into backlog/add-oauth2-oidc-corporate-authentication-to-backend
…iders Signed-off-by: Manuel Abascal <mjabascal10@gmail.com>
PLEASE READ BEFORE CONTINUING
To help us understand your contribution, please include the following in your pull request: