The main Bcrypt module should only contain the functions that the average developer would use, and all of the lower-level functions should be in the Bcrypt.Base module. With this in mind, the gen_salt function should really be in the Bcrypt.Base module, as most users will not use it directly.
I propose the following:
move the gen_salt function to the Bcrypt.Base module
deprecate the gen_salt function in the Bcrypt module (and remove it in the next major version)
riverrun
commented
2 years ago
The main
Bcryptmodule should only contain the functions that the average developer would use, and all of the lower-level functions should be in theBcrypt.Basemodule. With this in mind, thegen_saltfunction should really be in theBcrypt.Basemodule, as most users will not use it directly.I propose the following:
gen_saltfunction to theBcrypt.Basemodulegen_saltfunction in theBcryptmodule (and remove it in the next major version)