Skip to content

Enable Enter key submission for One-Time Token login#18824

Open
AnanthaKrishnanJ2001 wants to merge 1 commit intospring-projects:mainfrom
AnanthaKrishnanJ2001:fix/ott-login-enter-key-submission
Open

Enable Enter key submission for One-Time Token login#18824
AnanthaKrishnanJ2001 wants to merge 1 commit intospring-projects:mainfrom
AnanthaKrishnanJ2001:fix/ott-login-enter-key-submission

Conversation

@AnanthaKrishnanJ2001
Copy link

Description

This PR fixes a bug in the default login page where pressing "Enter" in the One-Time Token (OTT) username field would not submit the form. By moving the submit button inside the <form> tags and removing the redundant form attribute, we restore standard browser submission behavior.

Testing

  • Updated DefaultLoginPageGeneratingFilterTests to match the corrected HTML structure.
  • Verified that pressing Enter now triggers the submission.

Closes #18817

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Feb 28, 2026
@jzheaux jzheaux self-assigned this Mar 3, 2026
@jzheaux jzheaux added type: enhancement A general enhancement and removed status: waiting-for-triage An issue we've not yet triaged labels Mar 3, 2026
@jzheaux jzheaux changed the title Fix: Enable Enter key submission for One-Time Token login Enable Enter key submission for One-Time Token login Mar 3, 2026
@jzheaux jzheaux added in: web An issue in web modules (web, webmvc) labels Mar 3, 2026
@AnanthaKrishnanJ2001 AnanthaKrishnanJ2001 force-pushed the fix/ott-login-enter-key-submission branch from ceeab3c to 4af211f Compare March 3, 2026 02:47
Signed-off-by: Anantha Krishnan <ananthakrishnanj2001@gmail.com>
@AnanthaKrishnanJ2001 AnanthaKrishnanJ2001 force-pushed the fix/ott-login-enter-key-submission branch from 4af211f to 2a23f3b Compare March 3, 2026 02:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

in: web An issue in web modules (web, webmvc) type: enhancement A general enhancement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Enter Should Submit Request OTT Token

3 participants