[java-idp-plugin-duo] branch dev/JDUO-80 updated: Use two forms in the view so the return key works.

[Scott Cantor]

This is an automated email from the git hooks/post-receive script.

scantor pushed a commit to branch dev/JDUO-80
in repository java-idp-plugin-duo.

View the commit online:
http://git.shibboleth.net/view/?p=java-idp-plugin-duo.git;a=commit;h=3783c7ba2f9586228439a2bd8ea31bc7f9b9c3c9

The following commit(s) were added to refs/heads/dev/JDUO-80 by this push:
     new 3783c7ba Use two forms in the view so the return key works.
3783c7ba is described below

commit 3783c7ba2f9586228439a2bd8ea31bc7f9b9c3c9
Author: Scott Cantor <cantor.2 at osu.edu>
AuthorDate: Thu Jan 4 09:18:57 2024 -0500

    Use two forms in the view so the return key works.
---
 .../idp/plugin/authn/duo/views/passwordless.vm     | 58 ++++++++++++----------
 1 file changed, 31 insertions(+), 27 deletions(-)

diff --git a/idp-duo-impl/src/main/resources/net/shibboleth/idp/plugin/authn/duo/views/passwordless.vm b/idp-duo-impl/src/main/resources/net/shibboleth/idp/plugin/authn/duo/views/passwordless.vm
index d22a831f..2eb98fb8 100644
--- a/idp-duo-impl/src/main/resources/net/shibboleth/idp/plugin/authn/duo/views/passwordless.vm
+++ b/idp-duo-impl/src/main/resources/net/shibboleth/idp/plugin/authn/duo/views/passwordless.vm
@@ -45,44 +45,48 @@
             </header>
             
             <section>
-                <form action="$flowExecutionUrl" method="post">
-                    #parse("csrf/csrf.vm")
+                #*
+                //
+                //    SP Description & Logo (optional)
+                //    These idpui lines will display added information (if available
+                //    in the metadata) about the Service Provider (SP) that requested
+                //    authentication. These idpui lines are "active" in this example
+                //    (not commented out) - this extra SP info will be displayed.
+                //    Remove or comment out these lines to stop the display of the
+                //    added SP information.
+                //
+                *#
+                #set ($logo = $rpUIContext.getLogo())
+                #if ($logo)
+                    <img class="service-logo" src= "$encoder.encodeForHTMLAttribute($logo)" alt="$encoder.encodeForHTMLAttribute($serviceName)">
+                #end
+                #set ($desc = $rpUIContext.getServiceDescription())
+                #if ($desc)
+                    <p>$encoder.encodeForHTML($desc)</p>
+                #end
 
-                    #*
-                    //
-                    //    SP Description & Logo (optional)
-                    //    These idpui lines will display added information (if available
-                    //    in the metadata) about the Service Provider (SP) that requested
-                    //    authentication. These idpui lines are "active" in this example
-                    //    (not commented out) - this extra SP info will be displayed.
-                    //    Remove or comment out these lines to stop the display of the
-                    //    added SP information.
-                    //
-                    *#
-                    #set ($logo = $rpUIContext.getLogo())
-                    #if ($logo)
-                        <img class="service-logo" src= "$encoder.encodeForHTMLAttribute($logo)" alt="$encoder.encodeForHTMLAttribute($serviceName)">
-                    #end
-                    #set ($desc = $rpUIContext.getServiceDescription())
-                    #if ($desc)
-                        <p>$encoder.encodeForHTML($desc)</p>
-                    #end
+                <!-- Two forms are used below to allow the return key to trigger the Passwordless option after entering a name. -->
 
-                    <blockquote>#springMessageText("idp.duo.passwordless.explain", "If you've enrolled a passkey or device/token for passwordless login,
-                    please enter your username below and press the corresponding button. To bypass this option, just press the alternate button
-                    to perform a traditional login.")</blockquote>
+                <blockquote>#springMessageText("idp.duo.passwordless.explain", "If you've enrolled a passkey or device/token for passwordless login,
+                please enter your username below and press the corresponding button. To bypass this option, just press the alternate button
+                to perform a traditional login.")</blockquote>
 
+                <form action="$flowExecutionUrl" method="post">
+                    #parse("csrf/csrf.vm")
                     <div class="grid">
                         <div class="grid-item">
                             <button type="submit" name="_eventId_cancel"
                                 >#springMessageText("idp.duo.passwordless.cancel", "Login with Password")</button>
                         </div>
                     </div>
+                </form>
                     
-                    #if ($eventId == "RequestUnsupported")
-                        <p class="output-message output--error">$encoder.encodeForHTML("#springMessageText('idp.duo.passwordless.unsupported', 'You have not enrolled a qualifying device for Passwordless use.')")</p>
-                    #end
+                #if ($eventId == "RequestUnsupported")
+                    <p class="output-message output--error">$encoder.encodeForHTML("#springMessageText('idp.duo.passwordless.unsupported', 'You have not enrolled a qualifying device for Passwordless use.')")</p>
+                #end
                         
+                <form action="$flowExecutionUrl" method="post">
+                    #parse("csrf/csrf.vm")
                     <label for="username">#springMessageText("idp.login.username", "Username")</label>
                     <input id="username" name="j_username" type="text"
                         value="#if($username)$encoder.encodeForHTML($username)#end" />

-- 
To stop receiving notification emails like this one, please contact
the administrator of this repository.