Skip to content
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 Reactive One-Time Token Login Kotlin DSL Support #15888

Merged
merged 2 commits into from
Oct 15, 2024

Conversation

franticticktick
Copy link
Contributor

Closes gh-15887

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Oct 8, 2024
@jzheaux jzheaux added in: config An issue in spring-security-config type: enhancement A general enhancement and removed status: waiting-for-triage An issue we've not yet triaged labels Oct 14, 2024
@jzheaux jzheaux added this to the 6.4.0-RC1 milestone Oct 14, 2024
@jzheaux
Copy link
Contributor

jzheaux commented Oct 14, 2024

Thanks, @CrazyParanoid! While we are here, will you please do a quick review on the existing Kotlin documentation samples to ensure that they align with your changes? Some of them look like they should be adjusted.

@franticticktick
Copy link
Contributor Author

Hi @jzheaux, thanks for your commit. I'll fix the documentation, and I also see that I forgot to add the documentation for Kotlin here. I'll try to do it today.

@franticticktick
Copy link
Contributor Author

@jzheaux I tried to eliminate the consequences of renaming the API, perhaps I could have missed something somewhere. Could you please review this PR?

@jzheaux
Copy link
Contributor

jzheaux commented Oct 15, 2024

Thanks for the additional updates, @CrazyParanoid. Are you able to isolate the Kotlin DSL support changes to a separate commit from the rest? That way it's clear in the commit log what was needed for which feature.

The names of variables and methods have been adjusted in accordance with the names of the one-time token login API components.

Issue spring-projectsgh-15114
@jzheaux jzheaux self-assigned this Oct 15, 2024
@jzheaux jzheaux merged commit d37d41c into spring-projects:main Oct 15, 2024
6 checks passed
@jzheaux
Copy link
Contributor

jzheaux commented Oct 15, 2024

Thanks, @CrazyParanoid! This is now merged into main.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
in: config An issue in spring-security-config type: enhancement A general enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add Reactive One-Time Token Login Kotlin DSL Support
3 participants