[freeside-commits] branch master updated. 9acbc523636914be2788718d54f92dcc69863a28

[Jonathan Prykop]

The branch, master has been updated
       via  9acbc523636914be2788718d54f92dcc69863a28 (commit)
      from  3f15da8a0dd800432253294e623c6caaabc59fe3 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit 9acbc523636914be2788718d54f92dcc69863a28
Author: Jonathan Prykop <jonathan at freeside.biz>
Date:   Fri May 6 18:48:49 2016 -0500

    RT#41641: Disable strict password requirements [v4+ only, passwordmin default increased to 8]

diff --git a/FS/FS/Password_Mixin.pm b/FS/FS/Password_Mixin.pm
index 3dd9ce4..122e3fd 100644
--- a/FS/FS/Password_Mixin.pm
+++ b/FS/FS/Password_Mixin.pm
@@ -46,7 +46,7 @@ sub is_password_allowed {
   # basic checks using Data::Password;
   # options for Data::Password
   $DICTIONARY = 0;   # minimum length of disallowed words, false value disables dictionary checking
-  $MINLEN = $conf->config('passwordmin') || 6;
+  $MINLEN = $conf->config('passwordmin') || 8;
   $MAXLEN = $conf->config('passwordmax') || 12;
   $GROUPS = 4;       # must have all 4 'character groups': numbers, symbols, uppercase, lowercase
   # other options use the defaults listed below:
diff --git a/FS/FS/svc_acct.pm b/FS/FS/svc_acct.pm
index 93659f9..b7967ae 100644
--- a/FS/FS/svc_acct.pm
+++ b/FS/FS/svc_acct.pm
@@ -67,11 +67,11 @@ FS::UID->install_callback( sub {
   @shells = $conf->config('shells');
   $usernamemin = $conf->config('usernamemin') || 2;
   $usernamemax = $conf->config('usernamemax');
-  $passwordmin = $conf->config('passwordmin'); # || 6;
-  #blank->6, keep 0
+  $passwordmin = $conf->config('passwordmin');
+  #blank->8, keep 0
   $passwordmin = ( defined($passwordmin) && $passwordmin =~ /\d+/ )
                    ? $passwordmin
-                   : 6;
+                   : 8;
   $passwordmax = $conf->config('passwordmax') || 12;
   $username_letter = $conf->exists('username-letter');
   $username_letterfirst = $conf->exists('username-letterfirst');

-----------------------------------------------------------------------

Summary of changes:
 FS/FS/Password_Mixin.pm |    2 +-
 FS/FS/svc_acct.pm       |    6 +++---
 2 files changed, 4 insertions(+), 4 deletions(-)