Changeset 55094 for trunk/src/wp-admin/user-new.php

Timestamp:

01/19/2023 08:51:41 AM (3 years ago)

Author:

audrasjb

Message:

Login and Registration: Disable spellcheck for password fields.

This changeset adds spellcheck="false" attribute to various password fields.

The spellcheck global attribute defines whether the element may be checked for spelling errors. The false value indicates that the element should not be checked for spelling errors, which is relevant for a password field.

Furthermore, and as per MDN specs, using spellchecking can have consequences for users' security and privacy. The specification does not regulate how spellchecking is done and the content of the element may be sent to a third party for spellchecking results. Thus, it is recommended to set spellcheck attribute to false for elements that can contain sensitive information. Which is the case for password fields.

Props dziudek, audrasjb, gainesm, fosuahmed.
Fixes #56763.

File:

• trunk/src/wp-admin/user-new.php (modified) (2 diffs)

trunk/src/wp-admin/user-new.php

@@ -572 +572 @@
                 <?php $initial_password = wp_generate_password( 24 ); ?>
                 <span class="password-input-wrapper">
-                    <input type="password" name="pass1" id="pass1" class="regular-text" autocomplete="new-password" data-reveal="1" data-pw="<?php echo esc_attr( $initial_password ); ?>" aria-describedby="pass-strength-result" />
+                    <input type="password" name="pass1" id="pass1" class="regular-text" autocomplete="new-password" spellcheck="false" data-reveal="1" data-pw="<?php echo esc_attr( $initial_password ); ?>" aria-describedby="pass-strength-result" />
                 </span>
                 <button type="button" class="button wp-hide-pw hide-if-no-js" data-toggle="0" aria-label="<?php esc_attr_e( 'Hide password' ); ?>">
@@ -585 +585 @@
         <th scope="row"><label for="pass2"><?php _e( 'Repeat Password' ); ?> <span class="description"><?php _e( '(required)' ); ?></span></label></th>
         <td>
-        <input name="pass2" type="password" id="pass2" autocomplete="new-password" aria-describedby="pass2-desc" />
+        <input name="pass2" type="password" id="pass2" autocomplete="new-password" spellcheck="false" aria-describedby="pass2-desc" />
         <p class="description" id="pass2-desc"><?php _e( 'Type the password again.' ); ?></p>
         </td>