diff options
author | 2022-05-09 01:31:46 -0700 | |
---|---|---|
committer | 2022-05-09 10:31:46 +0200 | |
commit | b24b71c0a4ca9c86e1d5db12e9472c6ab1ecd5f5 (patch) | |
tree | 03f35f6f3fe2b2fa4cd26dae9fd820d355e16668 /internal/validate/formvalidation.go | |
parent | [bugfix] Fix remote media pruning failing if media already gone (#548) (diff) | |
download | gotosocial-b24b71c0a4ca9c86e1d5db12e9472c6ab1ecd5f5.tar.xz |
[feature] Include password strength in error message when password strength is too low (#550)
* When password validation fails, return how close to enough entropy it has.
* Shorter version of low-strength password error message
Diffstat (limited to 'internal/validate/formvalidation.go')
-rw-r--r-- | internal/validate/formvalidation.go | 12 |
1 files changed, 11 insertions, 1 deletions
diff --git a/internal/validate/formvalidation.go b/internal/validate/formvalidation.go index e4c169788..e0c27628b 100644 --- a/internal/validate/formvalidation.go +++ b/internal/validate/formvalidation.go @@ -22,6 +22,7 @@ import ( "errors" "fmt" "net/mail" + "strings" apimodel "github.com/superseriousbusiness/gotosocial/internal/api/model" "github.com/superseriousbusiness/gotosocial/internal/regexes" @@ -53,7 +54,16 @@ func NewPassword(password string) error { return fmt.Errorf("password should be no more than %d chars", maximumPasswordLength) } - return pwv.Validate(password, minimumPasswordEntropy) + if err := pwv.Validate(password, minimumPasswordEntropy); err != nil { + // Modify error message to include percentage requred entropy the password has + percent := int(100 * pwv.GetEntropy(password) / minimumPasswordEntropy) + return errors.New(strings.ReplaceAll( + err.Error(), + "insecure password", + fmt.Sprintf("password is %d%% strength", percent))) + } + + return nil // pasword OK } // Username makes sure that a given username is valid (ie., letters, numbers, underscores, check length). |