Skip to content

Conversation

@pablocarle
Copy link
Contributor

Description

OIDC provider configuration for the login flow supports two providers: okta and entra, the redirectUri parameter can break the startup of API ML if no redirectUri for the provider is found.
This change creates a default for any provider (current and future)

Type of change

  • fix: Bug fix (non-breaking change which fixes an issue)

Checklist:

  • My code follows the style guidelines of this project
  • PR title conforms to commit message guideline ## Commit Message Structure Guideline
  • I have commented my code, particularly in hard-to-understand areas. In JS I did provide JSDoc
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • The java tests in the area I was working on leverage @nested annotations
  • Any dependent changes have been merged and published in downstream modules

Signed-off-by: Pablo Carle <pablo.carle@broadcom.com>
Signed-off-by: Pablo Carle <pablo.carle@broadcom.com>
@sonarqubecloud
Copy link

@pablocarle pablocarle merged commit 644c9c9 into v3.x.x Sep 29, 2025
38 checks passed
@pablocarle pablocarle deleted the reboot/fix/provider-default branch September 29, 2025 07:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Development

Successfully merging this pull request may close these issues.

2 participants