Fresh installs should also follow updated password policy #1087

Closed
wants to merge 2 commits into
from

Conversation

Projects
None yet
2 participants
Contributor

randomecho commented Oct 31, 2011

Since new registrations and password updates are checked against 8 character minimums and alphanumeric, make the fresh install of a setup follow the same policy for consistency.

999 - Improve ThinkUp's password policy

Owner

ginatrapani commented Oct 31, 2011

Forgot about installation! Thanks so much.

Sent from my mobile
On Oct 31, 2011 12:50 PM, "Soon Van" <
reply@reply.github.com>
wrote:

Since new registrations and password updates are checked against 8
character minimums and alphanumeric, make the fresh install of a setup
follow the same policy for consistency.

999 - Improve ThinkUp's password policy

You can merge this Pull Request by running:

git pull https://github.com/randomecho/ThinkUp master

Or you can view, comment on it, or merge it online at:

ginatrapani#1087

-- Commit Summary --

  • test against new password policy on fresh installs
  • fresh installs should also conform to new password policy

-- File Changes --

M tests/TestOfInstallerController.php (58)
M webapp/_lib/controller/class.InstallerController.php (24)

-- Patch Links --

https://github.com/ginatrapani/ThinkUp/pull/1087.patch
https://github.com/ginatrapani/ThinkUp/pull/1087.diff

Reply to this email directly or view it on GitHub:
ginatrapani#1087

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment