@mattab opened this Issue on June 16th 2016 Owner

Currently, password is limited to maximum 80 characters. We should not limit max password length as it is not a standard practise. Follows up #6632 #4558

@blueelvis commented on July 11th 2016 Contributor

We need to define a theoretical limit. How about we use the constant PHP_INT_MAX and use that as the theoretical max?

Depending on the architecture maximum value would be -

32-bit builds of PHP: 2,147,483,647 (~ ± 2 billion)
64-bit builds of PHP: 9,223,372,036,854,775,807 (~ ± 9 quintillion)

@blueelvis commented on July 15th 2016 Contributor

@mattab - What say?

@mattab commented on July 18th 2016 Owner

@blueelvis since there will be other limits hit before we hit the 32-bit limit.. I think it's not needed to put in the limit and we can simply accept all strings as they come (while still enforcing the min password length)

@tsteur commented on August 3rd 2016 Owner

Kudos to @blueelvis

This Issue was closed on August 3rd 2016
Powered by GitHub Issue Mirror