-
Notifications
You must be signed in to change notification settings - Fork 134
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
Extend endpoint registration UI to support Client ID and Client Secret #2920
Conversation
Hey nwmac! Thanks for submitting this pull request! I'm here to inform the recipients of the pull request that you and the commit authors have already signed the CLA. |
Codecov Report
@@ Coverage Diff @@
## v2-master #2920 +/- ##
=============================================
- Coverage 46.73% 46.72% -0.02%
=============================================
Files 607 607
Lines 26169 26180 +11
Branches 5926 5930 +4
=============================================
+ Hits 12230 12232 +2
- Misses 13939 13948 +9 |
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.
LGTM, can merge once gates pass
- Show redirect_uri message underneath sso checkbox - Merge with new optional client id/secret section split these two components - Text for redirect_uri is now limited by form, alernative is to place SSO under optional section
This PR adds the ability to specify the Client ID and Client Secret to use when registering an endpoint.
The back-end changes are already in place for this, so this PR adds the necessary fields into the UI