Skip to content

Add Kotlin DSL for OAuth Authorization Server configuration#18150

Open
mehrdadbozorgmehr wants to merge 2 commits intospring-projects:mainfrom
mehrdadbozorgmehr:gh-18070
Open

Add Kotlin DSL for OAuth Authorization Server configuration#18150
mehrdadbozorgmehr wants to merge 2 commits intospring-projects:mainfrom
mehrdadbozorgmehr:gh-18070

Conversation

@mehrdadbozorgmehr
Copy link
Copy Markdown
Contributor

Closes gh-18070

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Nov 7, 2025
@jgrandja jgrandja self-assigned this Nov 10, 2025
@jgrandja jgrandja added type: enhancement A general enhancement in: oauth2 An issue in OAuth2 modules (oauth2-core, oauth2-client, oauth2-resource-server, oauth2-jose) and removed status: waiting-for-triage An issue we've not yet triaged labels Nov 10, 2025
@jgrandja jgrandja added this to the 7.1.0-M1 milestone Nov 10, 2025
@jgrandja
Copy link
Copy Markdown
Contributor

Thanks for the PR @mehrdadbozorgmehr.

I've scheduled this for 7.1.0-M1. I will review this soon.

@jgrandja jgrandja modified the milestones: 7.1.0-M1, 7.1.x Jan 19, 2026
@mehrdadbozorgmehr
Copy link
Copy Markdown
Contributor Author

@jgrandja I'm ready to finish it. :)

@jgrandja
Copy link
Copy Markdown
Contributor

jgrandja commented Feb 9, 2026

@mehrdadbozorgmehr Apologies for the delay.

I've been working on a few high priority items and still need more time to complete.

I do plan on getting this into 7.1 but I need more time until I can circle back to this to do a review.

Thanks for your patience.

@jgrandja
Copy link
Copy Markdown
Contributor

@mehrdadbozorgmehr Thank you for your patience.

I did a quick high-level review and noticed a few Kotlin DSL files are missing, e.g. OAuth2AuthorizationEndpointDsl, OAuth2TokenEndpointDsl, OAuth2ClientAuthenticationDsl, etc.

Also, please add more tests that make use of the Kotlin DSL, specifically, authorizationRequestConverter(), authorizationRequestConverters(), authenticationProvider(), authenticationProviders(), successResponseHandler(), errorResponseHandler(), etc.

I will do a full review after you submit the complete DSL. Thanks.

@mehrdadbozorgmehr
Copy link
Copy Markdown
Contributor Author

Hello,
Thanks for the review. Due to the recent war situation in Iran and widespread internet disruptions, I currently have very limited access to the open internet. I managed to connect with difficulty to send this message.
I appreciate your understanding and will apply the requested changes as soon as possible.(when access to internet immediately )

@jgrandja
Copy link
Copy Markdown
Contributor

jgrandja commented Apr 1, 2026

@mehrdadbozorgmehr I'm so sorry that you and your family have to deal with this war. Take care of yourself and your family and be safe.

If you would like, I can finish this off for you.

FYI, 7.1.0-RC1 is scheduled for April 20 so this would have to be merged before then otherwise it will need to wait for the 7.2 cycle. Please let me know your preference. Thank you.

@jgrandja jgrandja modified the milestones: 7.1.x, 7.2.x Apr 14, 2026
Signed-off-by: Mehrdad <mehrdad.bozorgmehr@gmail.com>
Signed-off-by: M.Bozorgmehr <mehrdad.bozorgmehr@gmail.com>
…dpoint, and Token Endpoint configuration

Signed-off-by: M.Bozorgmehr <mehrdad.bozorgmehr@gmail.com>
@mehrdadbozorgmehr
Copy link
Copy Markdown
Contributor Author

@jgrandja Thank you so much for your kind words, I really appreciate it. Things are more stable now and I’ve been able to address the requested changes.

I’ve just pushed the updates—please let me know if anything else is needed. And thanks again for your support 🙏

@mehrdadbozorgmehr I'm so sorry that you and your family have to deal with this war. Take care of yourself and your family and be safe.

If you would like, I can finish this off for you.

FYI, 7.1.0-RC1 is scheduled for April 20 so this would have to be merged before then otherwise it will need to wait for the 7.2 cycle. Please let me know your preference. Thank you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

in: oauth2 An issue in OAuth2 modules (oauth2-core, oauth2-client, oauth2-resource-server, oauth2-jose) type: enhancement A general enhancement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add Kotlin DSL for authorization server

3 participants