Skip to content

Return registration_endpoint in OidcProviderConfigurationEndpointFilter #881

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

Conversation

sahariardev
Copy link
Contributor

Before: client registration endpoint was not retuned in oidc
Provider Configuration response

After: Returns client registration endpoint in oidcprovider configuration
response if client registration is enabled

Fixes gh-370

@sahariardev
Copy link
Contributor Author

Old PR link #762

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Sep 2, 2022
@sahariardev sahariardev force-pushed the return_registration_endpoint_in_oidcProviderConfigurationEndpointFilter branch 2 times, most recently from 2db51fc to 3e90c2c Compare September 3, 2022 10:29
@sahariardev
Copy link
Contributor Author

Hi @jgrandja,
I have done all the changes. Please have a look.

@jgrandja jgrandja added status: duplicate A duplicate of another issue type: enhancement A general enhancement and removed status: waiting-for-triage An issue we've not yet triaged labels Sep 7, 2022
@jgrandja jgrandja self-assigned this Sep 7, 2022
Copy link
Collaborator

@jgrandja jgrandja left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the updates @sahariardev. Please see review comments.

Before: client registration endpoint was not retuned in oidc
Provider Configuration response

After: Returns client registration endpoint in oidcprovider configuration
response if client registration is enabled

Fixes spring-projectsgh-370
@sahariardev sahariardev force-pushed the return_registration_endpoint_in_oidcProviderConfigurationEndpointFilter branch from 3e90c2c to f1eb744 Compare September 9, 2022 12:55
@sahariardev
Copy link
Contributor Author

Hi @jgrandja,
I have done all the changes.
Thanks.

@jgrandja jgrandja added this to the 0.4.0-M2 milestone Sep 20, 2022
jgrandja added a commit that referenced this pull request Sep 20, 2022
@jgrandja
Copy link
Collaborator

Thanks for all the updates @sahariardev. This is now merged.

FYI, I added a polish commit with some minor updates.
Thanks again!

@jgrandja jgrandja closed this Sep 20, 2022
jgrandja added a commit that referenced this pull request Sep 20, 2022
The following commits are merged using the default merge strategy.

80b0185 Update README with documentation links
4d94e70 Decompose OAuth2AuthorizationCodeRequestAuthenticationProvider
cd6f1d7 Return registration_endpoint when client registration is enabled
26aed3c Polish gh-881
92dbcf2 Move integration tests for OidcProviderConfiguration
doba16 pushed a commit to doba16/spring-authorization-server that referenced this pull request Apr 21, 2023
doba16 pushed a commit to doba16/spring-authorization-server that referenced this pull request Apr 21, 2023
The following commits are merged using the default merge strategy.

ccdd8ce Update README with documentation links
c70255a Decompose OAuth2AuthorizationCodeRequestAuthenticationProvider
8d51722 Return registration_endpoint when client registration is enabled
fdf1200 Polish spring-projectsgh-881
aee5613 Move integration tests for OidcProviderConfiguration
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: duplicate A duplicate of another issue type: enhancement A general enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants