-
Notifications
You must be signed in to change notification settings - Fork 544
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
Introduce New Templates for Custom Authenticators #6226
base: master
Are you sure you want to change the base?
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #6226 +/- ##
============================================
+ Coverage 45.66% 45.74% +0.07%
- Complexity 14040 14263 +223
============================================
Files 1632 1636 +4
Lines 100532 102402 +1870
Branches 17421 16822 -599
============================================
+ Hits 45911 46846 +935
- Misses 47931 48795 +864
- Partials 6690 6761 +71
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
...ntity.extension.mgt.feature/resources/extensions/connections/custom-authentication/info.json
Outdated
Show resolved
Hide resolved
...nsion.mgt.feature/resources/extensions/connections/twofactor-custom-authentication/info.json
Outdated
Show resolved
Hide resolved
"name": "Custom Authentication", | ||
"description": "Enable login for users with external authentication service.", | ||
"image": "", | ||
"category": "DEFAULT", |
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.
What is meant by this param. What are the other available categories
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.
Currently, all the available connections fall under the DEFAULT category. If we are introducing a new category with custom authentication, we would have to initiate that separately.
"description": "Enable login for users with external authentication service.", | ||
"image": "", | ||
"category": "DEFAULT", | ||
"displayOrder": -1, |
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.
What is meant by -1. What is purpose of this parameter
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.
This is to represent the order in which the card will be displayed on the connections page. -1 implied that this will not be displayed and an updated grouped template will be displayed instead.
...n.mgt.feature/resources/extensions/connections/twofactor-custom-authentication/template.json
Show resolved
Hide resolved
Quality Gate passedIssues Measures |
Proposed changes in this pull request
This PR introduces new JSON files related to custom authenticators. Three types of custom authenticators are introduced which will be grouped under a generic custom-authentication template.
Following are the template-ids of the newly added templates.
The following is the proposed UI design that'd display the rendered content. This will be added to the product as a new connection.
Related Emails:
[Architecture] Facilitate user authentication via a external service (custom authentication extension)
Related Issue
When should this PR be merged
[Please describe any preconditions that need to be addressed before we
can merge this pull request.]
Follow up actions
[List any possible follow-up actions here; for instance, testing data
migrations, software that we need to install on staging and production
environments.]
Checklist (for reviewing)
General
Functionality
Code
Tests
Security
Documentation