diff options
author | Christopher Speller <crspeller@gmail.com> | 2015-10-20 10:38:15 -0400 |
---|---|---|
committer | Christopher Speller <crspeller@gmail.com> | 2015-10-20 10:38:15 -0400 |
commit | 98aaf9067b362e104fae29e0ae0a57a50896a626 (patch) | |
tree | 12f9fdc49595e427339d08c41a659311308e426d | |
parent | 7d3bc099b969ebac7c3f9a9eaab67a4aa7f4707a (diff) | |
parent | f7e51b14d7a05cef38e3b439cef9d392e8ebee7d (diff) | |
download | chat-98aaf9067b362e104fae29e0ae0a57a50896a626.tar.gz chat-98aaf9067b362e104fae29e0ae0a57a50896a626.tar.bz2 chat-98aaf9067b362e104fae29e0ae0a57a50896a626.zip |
Merge pull request #1115 from mattermost/salt-fix
Allowing salts longer than 32 chars
-rw-r--r-- | model/config.go | 16 |
1 files changed, 8 insertions, 8 deletions
diff --git a/model/config.go b/model/config.go index 8a11b7bb7..59caf0884 100644 --- a/model/config.go +++ b/model/config.go @@ -184,8 +184,8 @@ func (o *Config) IsValid() *AppError { return NewAppError("Config.IsValid", "Invalid maximum users per team for team settings. Must be a positive number.", "") } - if len(o.SqlSettings.AtRestEncryptKey) != 32 { - return NewAppError("Config.IsValid", "Invalid at rest encrypt key for SQL settings. Must be 32 chars.", "") + if len(o.SqlSettings.AtRestEncryptKey) < 32 { + return NewAppError("Config.IsValid", "Invalid at rest encrypt key for SQL settings. Must be 32 chars or more.", "") } if !(o.SqlSettings.DriverName == DATABASE_DRIVER_MYSQL || o.SqlSettings.DriverName == DATABASE_DRIVER_POSTGRES) { @@ -232,20 +232,20 @@ func (o *Config) IsValid() *AppError { return NewAppError("Config.IsValid", "Invalid thumbnail width for file settings. Must be a positive number.", "") } - if len(o.FileSettings.PublicLinkSalt) != 32 { - return NewAppError("Config.IsValid", "Invalid public link salt for file settings. Must be 32 chars.", "") + if len(o.FileSettings.PublicLinkSalt) < 32 { + return NewAppError("Config.IsValid", "Invalid public link salt for file settings. Must be 32 chars or more.", "") } if !(o.EmailSettings.ConnectionSecurity == CONN_SECURITY_NONE || o.EmailSettings.ConnectionSecurity == CONN_SECURITY_TLS || o.EmailSettings.ConnectionSecurity == CONN_SECURITY_STARTTLS) { return NewAppError("Config.IsValid", "Invalid connection security for email settings. Must be '', 'TLS', or 'STARTTLS'", "") } - if len(o.EmailSettings.InviteSalt) != 32 { - return NewAppError("Config.IsValid", "Invalid invite salt for email settings. Must be 32 chars.", "") + if len(o.EmailSettings.InviteSalt) < 32 { + return NewAppError("Config.IsValid", "Invalid invite salt for email settings. Must be 32 chars or more.", "") } - if len(o.EmailSettings.PasswordResetSalt) != 32 { - return NewAppError("Config.IsValid", "Invalid password reset salt for email settings. Must be 32 chars.", "") + if len(o.EmailSettings.PasswordResetSalt) < 32 { + return NewAppError("Config.IsValid", "Invalid password reset salt for email settings. Must be 32 chars or more.", "") } if o.RateLimitSettings.MemoryStoreSize <= 0 { |