Hi all,

One of the browsers I test Drupal with won't get passed the Login page.
Drupal is sending the attribute: maxlength="" in the password input tag and the browser assumes that 0 is the allowed length and stops the user from typing anything into the password field.

Whilst, technically, it's a bug in the browser, as you only support passwords up to 32 characters, this patch provides the password input tag with the maxlength attribute set to 32.

The attached patch has been tested and works OK on my setup.

Yours,

Phil L.

CommentFileSizeAuthor
modules-user-module.diff443 bytesphilipnet

Comments

Anonymous’s picture

Anonymous (not verified) commented
Status: Needs review » Needs work

One of the browsers I test Drupal with won't get passed the Login page.

which browser?

Whilst, technically, it's a bug in the browser, as you only support passwords up to 32 characters, this patch provides the password input tag with the maxlength attribute set to 32.

Drupal supports passwords longer than 32 characters. the password value is turned into an md5 hash before its stored, which is why its 32 characters long in the database.

if we limit the form field to 32 characters we are introducing a limit that doesn't exist now, which is not a good thing. i think you need to find another solution to this problem.

wpd’s picture

wpd commented

Do not know if this will help your problem, but the validator I am using complained about the empty maxlength attribute.
Here is patch to fix the drupal code, so the empty attribute is not generated.
http://drupal.org/node/80749

killes@www.drop.org’s picture

killes@www.drop.org commented
Status: Needs work » Closed (duplicate)