vb.org Archive

vb.org Archive (https://vborg.vbsupport.ru/index.php)
-   vBulletin 4.x Add-ons (https://vborg.vbsupport.ru/forumdisplay.php?f=245)
-   -   Miscellaneous Hacks - Minimum Password Length (https://vborg.vbsupport.ru/showthread.php?t=264515)

Eric 05-30-2011 10:00 PM

Minimum Password Length
 
1 Attachment(s)
What is this?
This mod will allow you to force user passwords to be at least a certain length.


Features
  • Force minimum length on:
    • Registration
    • Edit Password
    • Reset Password

I've only tested this mod on vB 4.1.4/4.1.5 (alpha). It should work with previous versions, however I am not sure. If it works for you on an older version, let me know.


Installation
1. Download the `product-password_minlength.xml` file. (* may differ in name based on version)
2. Enter your AdminCP and go to Plugins & Products > Manage Products > [Add/Import Product]
3. Import the product using the `product-password_minlength.xml` file. (* may differ in name based on version)
4. Configure the mod in AdminCP -> Settings -> Options -> User Registration Options


Upgrading
In many cases, all you'll need to do to upgrade is follow the installation instructions above, but set "Allow Overwrite" to "Yes".


Changelog
Version 1.0.2, 07/05/2011
  • Changed the "Check Method" choice from a drop down to radio buttons (Boofo ;) )
  • Changed how the "UserId" "Check Method" works - it now is used for escluding User ID's
  • Fixed a bug in the plugin for updating profile - was not checking if a new password had been entered.

Version 1.0.1, 06/07/2011
  • Introduced three new options and one new plugin.
  • The new options are based around a "Check Method". You can choose to enforce the min. password length by userid, usergroup, or 'none' (all).

Version 1.0.0, 05/31/2011
  • Initial release.

Eric 05-31-2011 06:54 PM

Reserved.

Special Pages 05-31-2011 09:31 PM

Thank you Eric! :D

Lynne 05-31-2011 11:14 PM

Nice idea, Eric!

just.b.jealous 06-01-2011 12:58 AM

Works on vB 4.1.3,.. I did notice while importing it- that it gave an error of some sort but it finished importing to quickly before I had a chance to actually read the error. Everyting seems to be working fine though. Thanks, marked "Installed".

Eric 06-01-2011 01:02 AM

Quote:

Originally Posted by Lynne (Post 2202258)
Nice idea, Eric!

Thanks Lynne. :) I've seen a few folks request this several times so I finally decided to give it a go. I also thought it would be something useful given what is happening with passwords etc recently :)

Quote:

Originally Posted by just.b.jealous (Post 2202279)
Works on vB 4.1.3,.. I did notice while importing it- that it gave an error of some sort but it finished importing to quickly before I had a chance to actually read the error. Everyting seems to be working fine though. Thanks, marked "Installed".

That is odd. I will see if I can get my hands on 4.1.3 and see what that error might have been. There is not really anything in the file that should cause an error. :/

sulasno 06-01-2011 01:37 AM

tagged and thanks

can the mod dictate that a minimum of 1 Capital letter and I Digit must be used ?

vglobal 06-01-2011 04:45 AM

Tag for future. It would be great if we have a complex password mod.

Thanks

Boofo 06-01-2011 06:49 AM

Excellent idea, sir. ;)

Boofo 06-01-2011 08:54 AM

What is a good default setting for the length? I think 14 might be a little too long for some users to accept without whining. ;)

Also, I saw no error on importing the product on 4.1.3. Maybe another mod was not playing nice with the OP setup.

Eric 06-01-2011 10:59 AM

Quote:

Originally Posted by sulasno (Post 2202289)
tagged and thanks

can the mod dictate that a minimum of 1 Capital letter and I Digit must be used ?

Quote:

Originally Posted by vglobal (Post 2202325)
Tag for future. It would be great if we have a complex password mod.

Thanks

It is not possible to do that with this mod... at least, not yet. I will see what I can do. :)

Quote:

Originally Posted by Boofo (Post 2202342)
Excellent idea, sir. ;)

Thank you :)
Quote:

Originally Posted by Boofo (Post 2202357)
What is a good default setting for the length? I think 14 might be a little too long for some users to accept without whining. ;)

Also, I saw no error on importing the product on 4.1.3. Maybe another mod was not playing nice with the OP setup.

A good, secure, password is typically 12-16 (roughly) characters. But, I can understand some users having difficulty with that. I would say a good compromise would be 8 characters.

As for 4.1.3, that is what I was thinking - that maybe another mod was conflicting with it. Hopefully it is not an error with this mod itself. :)

Boofo 06-01-2011 04:59 PM

I compromised and set it at 10.

I didn't see anything in the code that would cause an error on import. I wouldn't worry about it unless you get anyone else having the same issues.

I would suggest maybe adding a setting for certain userids that could bypass the length check.

Eric 06-02-2011 04:23 PM

Quote:

Originally Posted by Boofo (Post 2202432)
I compromised and set it at 10.

I didn't see anything in the code that would cause an error on import. I wouldn't worry about it unless you get anyone else having the same issues.

I would suggest maybe adding a setting for certain userids that could bypass the length check.

That is a good idea Boofo, will implement it in the next release.

BirdOPrey5 06-04-2011 01:05 PM

Quote:

Originally Posted by Boofo (Post 2202432)
I compromised and set it at 10.

I didn't see anything in the code that would cause an error on import. I wouldn't worry about it unless you get anyone else having the same issues.

I would suggest maybe adding a setting for certain userids that could bypass the length check.

I suggest if possible add a feature to this mod to enforce minimum lengths on mod and admin accounts only.

Honestly it is extremely unlikely I wold join a forum requiring me to have a password over 6 to 8 characters.

Because... unless I'm a mod or admin, it's JUST a forum. NO ONE cares about my account and I care even less. So what someone cracks my password? Very unlikely on vBulletin where you can't brute-force your way in because it will lock you out after a few bad tries... I'm not going to jump through hoops to join a forum unless they are the only forum in their niche- and I know most admins can't claim that.

Just my opinion.

Boofo 06-04-2011 01:27 PM

Quote:

Originally Posted by BirdOPrey5 (Post 2203393)
I suggest if possible add a feature to this mod to enforce minimum lengths on mod and admin accounts only.

Honestly it is extremely unlikely I wold join a forum requiring me to have a password over 6 to 8 characters.

Because... unless I'm a mod or admin, it's JUST a forum. NO ONE cares about my account and I care even less. So what someone cracks my password? Very unlikely on vBulletin where you can't brute-force your way in because it will lock you out after a few bad tries... I'm not going to jump through hoops to join a forum unless they are the only forum in their niche- and I know most admins can't claim that.

Just my opinion.

I totally disagree.

Eric 06-04-2011 03:53 PM

Quote:

Originally Posted by BirdOPrey5 (Post 2203393)
I suggest if possible add a feature to this mod to enforce minimum lengths on mod and admin accounts only.

Honestly it is extremely unlikely I wold join a forum requiring me to have a password over 6 to 8 characters.

Because... unless I'm a mod or admin, it's JUST a forum. NO ONE cares about my account and I care even less. So what someone cracks my password? Very unlikely on vBulletin where you can't brute-force your way in because it will lock you out after a few bad tries... I'm not going to jump through hoops to join a forum unless they are the only forum in their niche- and I know most admins can't claim that.

Just my opinion.

I would disagree, actually. I think every member should have as secure a password as possible. These days when you have things like KeePass, etc - and browsers that will save the password... what is an extra 2-3 characters? Besides, the limit in this mod is configurable.

I may add a usergroup option though, we'll see. :)

Boofo 06-04-2011 04:01 PM

Quote:

Originally Posted by Eric (Post 2203463)
I would disagree, actually. I think every member should have as secure a password as possible. These days when you have things like KeePass, etc - and browsers that will save the password... what is an extra 2-3 characters? Besides, the limit in this mod is configurable.

I may add a usergroup option though, we'll see. :)

I think a userid option would be better. ;)

BirdOPrey5 06-04-2011 04:24 PM

Well obviously it's your mod... I'm just saying I think putting a 10 or 14 character minimum on regular user account on most forums is like putting a bank vault door on an empty shed in a rural area... Yeah it's more protection, but for what?

You have to balance security vs. the user experience and most forums don't need this type of security on their standard accounts. Admins need to realize IMO most of their sites aren't all that important in the scheme of things. If it was a bank account or medical history then yeah, by all means, enforce strong passwords... but a forum to talk about cars or art or video games? I'd be more concerned about frustrating new and existing members with password requirements far surpassing any bank account I've ever used and having them stop coming.

I use KeePass myself but I'm not going to go through the effort of making a new entry for every single forum I'm a member of. LOL.

Anyway, my suggestion is an option to enforce for mods and admins only... all other opinions aside.

jgt58 06-04-2011 05:58 PM

Quote:

Originally Posted by Boofo (Post 2203465)
I think a userid option would be better. ;)

This would be great if you could enforce this on a usergroup basis and not the regular members

Boofo 06-04-2011 06:13 PM

Quote:

Originally Posted by jgt58 (Post 2203522)
This would be great if you could enforce this on a usergroup basis and not the regular members

That makes absolutely no sense. Why even use it then?

Boofo 06-04-2011 06:14 PM

Quote:

Originally Posted by BirdOPrey5 (Post 2203476)
Well obviously it's your mod... I'm just saying I think putting a 10 or 14 character minimum on regular user account on most forums is like putting a bank vault door on an empty shed in a rural area... Yeah it's more protection, but for what?

You have to balance security vs. the user experience and most forums don't need this type of security on their standard accounts. Admins need to realize IMO most of their sites aren't all that important in the scheme of things. If it was a bank account or medical history then yeah, by all means, enforce strong passwords... but a forum to talk about cars or art or video games? I'd be more concerned about frustrating new and existing members with password requirements far surpassing any bank account I've ever used and having them stop coming.

I use KeePass myself but I'm not going to go through the effort of making a new entry for every single forum I'm a member of. LOL.

Anyway, my suggestion is an option to enforce for mods and admins only... all other opinions aside.

Not everyone feels their forums or members security are as unimportant as you feel they are.

BirdOPrey5 06-04-2011 10:35 PM

Quote:

Originally Posted by Boofo (Post 2203525)
That makes absolutely no sense. Why even use it then?

Because regular users have "no powers." If someone hacked a regular user account worst thing they could do is post as them... So what if that happens?

Mod and Admin accounts however need to be protected for the security of the forum and the protection of member's private info.

Eric 06-07-2011 06:35 PM

Version 1.0.1, 06/07/2011
  • Introduced three new options and one new plugin.
  • The new options are based around a "Check Method". You can choose to enforce the min. password length by userid, usergroup, or 'none' (all).

Boofo 06-08-2011 01:27 AM

Thanks for the update. The only thing I would suggest is changing the "Minimum Password Length: Check Method" option to radio:piped instead of select:piped. And I would have excluded userids instead of including them.

Eric 06-08-2011 10:19 PM

Quote:

Originally Posted by Boofo (Post 2204861)
Thanks for the update. The only thing I would suggest is changing the "Minimum Password Length: Check Method" option to radio:piped instead of select:piped. And I would have excluded userids instead of including them.

Why change to the radio:piped?

And for the userids, that is what I had initially and tbh, don't even remember why I thought it should be changed - would not take much to change it back.

Boofo 06-08-2011 10:55 PM

Quote:

Originally Posted by Eric (Post 2205270)
Why change to the radio:piped?

A coding preference, I guess, as well as it shows all options instead of having to scroll through a drop-down box.

Quote:

Originally Posted by Eric (Post 2205270)
And for the userids, that is what I had initially and tbh, don't even remember why I thought it should be changed - would not take much to change it back.

I was wondering if maybe it was a simple mistake on your end. ;)

jgt58 06-09-2011 10:53 PM

Quote:

Originally Posted by Boofo (Post 2203525)
That makes absolutely no sense. Why even use it then?

Because to enforce staff having a more secure password than the normal users. Extra security is really not needed for normal users. If they are concerned about that , they will have a strong password. I WANT my staff to have a secure password , but there is no way to enforce that. This would be perfect with tweeks.

So yes , it does make sense :-)

Boofo 06-10-2011 01:36 AM

To you, maybe. I think my users are just as important as the staff and therefore should be given the same concern. Having their accounts hacked could be just as disastrous, if not more so, than any staff members.

just.b.jealous 06-10-2011 04:03 AM

You should require it for admins/moderators and not regular users, trust me- they dislike it. But then again, any secure-minded admin already has a long enough, difficult to guess password. HAd this installed but users couldn't actually register- they all kept getting a "password doesn't contain required amount of characters, please try again" error, or something to that effect. Ending up having to disable it for the time being.

Eric 06-10-2011 06:13 AM

Quote:

Originally Posted by just.b.jealous (Post 2205695)
You should require it for admins/moderators and not regular users, trust me- they dislike it. But then again, any secure-minded admin already has a long enough, difficult to guess password. HAd this installed but users couldn't actually register- they all kept getting a "password doesn't contain required amount of characters, please try again" error, or something to that effect. Ending up having to disable it for the time being.

I've tested this mod several times across 4.1.3 and 4.1.4 - works fine. You sure they actually were meeting the requirement? ;)

ninjadawg 06-10-2011 11:33 PM

This hack works on VB 4.1.1

Nice work thanks

ND

Boofo 07-04-2011 04:55 AM

A member tried to change their email address tonight and they got this error:

Quote:

The password you entered is not long enough. Your password needs to be at least 10 characters in length.

The password they use is 11 characters. They tried it three times and kept getting the error. I have it set to 10.

Eric 07-05-2011 09:42 AM

Quote:

Originally Posted by Boofo (Post 2216490)
A member tried to change their email address tonight and they got this error:




The password they use is 11 characters. They tried it three times and kept getting the error. I have it set to 10.

Hmm, I think I see a cause for this - were they even trying to update their password, or just email?

Eric 07-05-2011 10:14 AM

Version 1.0.2, 07/05/2011
  • Changed the "Check Method" choice from a drop down to radio buttons (Boofo ;) )
  • Changed how the "UserId" "Check Method" works - it now is used for escluding User ID's
  • Fixed a bug in the plugin for updating profile - was not checking if a new password had been entered.

Boofo 07-05-2011 04:02 PM

Quote:

Originally Posted by Eric (Post 2216960)
Hmm, I think I see a cause for this - were they even trying to update their password, or just email?

No, I think they were just trying to update their email.

Eric 07-05-2011 04:04 PM

Quote:

Originally Posted by Boofo (Post 2217059)
No, I think they were just trying to update their email.

Ah, that is what I assumed. The update should fix that. :)

Boofo 07-05-2011 04:05 PM

Okay, thanks. ;)

OldSchoolDSL 11-25-2011 02:37 PM

Installed and working

4.1.8

cosy 04-14-2012 09:07 PM

this should be default for vbulletin

pczone 05-22-2012 07:16 AM

4.1.12

Installed and working

Thanks


All times are GMT. The time now is 05:05 AM.

Powered by vBulletin® Version 3.8.12 by vBS
Copyright ©2000 - 2025, vBulletin Solutions Inc.

X vBulletin 3.8.12 by vBS Debug Information
  • Page Generation 0.01447 seconds
  • Memory Usage 1,848KB
  • Queries Executed 10 (?)
More Information
Template Usage:
  • (1)ad_footer_end
  • (1)ad_footer_start
  • (1)ad_header_end
  • (1)ad_header_logo
  • (1)ad_navbar_below
  • (24)bbcode_quote_printable
  • (1)footer
  • (1)gobutton
  • (1)header
  • (1)headinclude
  • (6)option
  • (1)pagenav
  • (1)pagenav_curpage
  • (1)pagenav_pagelink
  • (1)post_thanks_navbar_search
  • (1)printthread
  • (40)printthreadbit
  • (1)spacer_close
  • (1)spacer_open 

Phrase Groups Available:
  • global
  • postbit
  • showthread
Included Files:
  • ./printthread.php
  • ./global.php
  • ./includes/init.php
  • ./includes/class_core.php
  • ./includes/config.php
  • ./includes/functions.php
  • ./includes/class_hook.php
  • ./includes/modsystem_functions.php
  • ./includes/class_bbcode_alt.php
  • ./includes/class_bbcode.php
  • ./includes/functions_bigthree.php 

Hooks Called:
  • init_startup
  • init_startup_session_setup_start
  • init_startup_session_setup_complete
  • cache_permissions
  • fetch_threadinfo_query
  • fetch_threadinfo
  • fetch_foruminfo
  • style_fetch
  • cache_templates
  • global_start
  • parse_templates
  • global_setup_complete
  • printthread_start
  • pagenav_page
  • pagenav_complete
  • bbcode_fetch_tags
  • bbcode_create
  • bbcode_parse_start
  • bbcode_parse_complete_precache
  • bbcode_parse_complete
  • printthread_post
  • printthread_complete