-
Notifications
You must be signed in to change notification settings - Fork 545
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
Add new authenticator configs for user defined authentication extensions. #6108
Add new authenticator configs for user defined authentication extensions. #6108
Conversation
1e739c9
to
b57cbda
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #6108 +/- ##
============================================
- Coverage 40.98% 40.76% -0.22%
+ Complexity 14698 14485 -213
============================================
Files 1764 1768 +4
Lines 119164 117713 -1451
Branches 20860 20305 -555
============================================
- Hits 48836 47984 -852
+ Misses 63008 62451 -557
+ Partials 7320 7278 -42
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
b57cbda
to
020a81a
Compare
...va/org/wso2/carbon/identity/application/common/model/AuthenticatorEndpointConfiguration.java
Outdated
Show resolved
Hide resolved
...va/org/wso2/carbon/identity/application/common/model/AuthenticatorEndpointConfiguration.java
Outdated
Show resolved
Hide resolved
...va/org/wso2/carbon/identity/application/common/model/AuthenticatorEndpointConfiguration.java
Outdated
Show resolved
Hide resolved
...g/wso2/carbon/identity/application/common/model/UserDefinedFederatedAuthenticatorConfig.java
Show resolved
Hide resolved
...a/org/wso2/carbon/identity/application/common/model/UserDefinedLocalAuthenticatorConfig.java
Show resolved
Hide resolved
...va/org/wso2/carbon/identity/application/common/model/AuthenticatorEndpointConfiguration.java
Outdated
Show resolved
Hide resolved
326d228
to
44bf32f
Compare
44bf32f
to
453f02d
Compare
...anagement/src/main/java/org/wso2/carbon/identity/action/management/model/Authentication.java
Outdated
Show resolved
Hide resolved
...anagement/src/main/java/org/wso2/carbon/identity/action/management/model/Authentication.java
Outdated
Show resolved
Hide resolved
...rg/wso2/carbon/identity/application/common/model/UserDefinedAuthenticatorEndpointConfig.java
Outdated
Show resolved
Hide resolved
...g/wso2/carbon/identity/application/common/model/UserDefinedFederatedAuthenticatorConfig.java
Outdated
Show resolved
Hide resolved
...g/wso2/carbon/identity/application/common/model/UserDefinedFederatedAuthenticatorConfig.java
Outdated
Show resolved
Hide resolved
...a/org/wso2/carbon/identity/application/common/model/UserDefinedLocalAuthenticatorConfig.java
Outdated
Show resolved
Hide resolved
...a/org/wso2/carbon/identity/application/common/model/UserDefinedLocalAuthenticatorConfig.java
Outdated
Show resolved
Hide resolved
52c9d10
to
649809f
Compare
649809f
to
0c77d92
Compare
Quality Gate passedIssues Measures |
...anagement/src/main/java/org/wso2/carbon/identity/action/management/model/Authentication.java
Show resolved
Hide resolved
PR builder started |
PR builder completed |
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.
Approving the pull request based on the successful pr build https://github.com/wso2/product-is/actions/runs/11735720931
PR builder started |
PR builder completed |
Issue:
With this PR, following changes will be added.
UserDefinedFederatedAuthenticator
model to for user defined fed authenticators extendingFederatedAuthenticatorConfig
class.UserDefinedLocalAuthenticator
model to for user defined fed authenticators extendingLocalAuthenticatorConfig
class.