FusionAuth / fusionauth-issues

FusionAuth issue submission project
https://fusionauth.io
90 stars 12 forks source link

Enforce bcrypt password length limit regardless of tenant configuration when using Bcrypt #2671

Closed robotdan closed 2 months ago

robotdan commented 4 months ago

What happened?

For the cases where you may have some users on bcrypt even though it isn't your default configuration we should validate these passwords separate from the primary configuration to enforce the max bcrypt length if it is less than the global config.

For example, if you set the default to 256 this is fine for most algorithms, but for Bcrypt we have to limit the password length to 50 bytes to be safe. In practice we could validate the byte length and not the character length, but this may be more difficult to communicate to the end user when they password is too long.

This is all possible today, but requires you to set this length limit for everyone even if some users are using a different algorithm.

Related

Version

1.48.0

Affects Versions

All

Documentation

robotdan commented 3 months ago

Internal:

andrewpai commented 2 months ago

Shipping in 1.50.0.