Item11705: Bulk registration crashes with uninitialized variable if Password column provided, and other failures.

pencil
Priority: Normal
Current State: Closed
Released In: 2.0.0
Target Release: major
Applies To: Engine
Component: BulkRegistration, Documentation, FoswikiUIRegister
Branches: trunk
Reported By: GeorgeClark
Waiting For:
Last Change By: GeorgeClark
If a Password is provided, the code compares $row->{Password} to $row->{Confirm}, and crashes.

  • If using bulk registration it doesn't make sense to confirm both passwords, so temporarily disable confirmation.
  • Use of the Password column is not documented.
  • Other validations are logged as fail, but actually ignored. User is still registered if minimum password length test fails.

See Support.Question1064

-- GeorgeClark - 29 Mar 2012

Ugh. The ManageDotPm::verify_bulkRegister tests don't really validate much. No checking that users were actually created, or that the log was created without errors.

-- GeorgeClark - 18 Jun 2014

The issue with ignored validation failures. "return" in a catch block doesn't return to the calling routine, but continues on in-line.

-- GeorgeClark - 18 Jun 2014
 

ItemTemplate edit

Summary Bulk registration crashes with uninitialized variable if Password column provided, and other failures.
ReportedBy GeorgeClark
Codebase 1.1.5 RC1, trunk
SVN Range
AppliesTo Engine
Component BulkRegistration, Documentation, FoswikiUIRegister
Priority Normal
CurrentState Closed
WaitingFor
Checkins distro:18eb5cfc8d4d
TargetRelease major
ReleasedIn 2.0.0
CheckinsOnBranches trunk
trunkCheckins distro:18eb5cfc8d4d
Release01x01Checkins
Topic revision: r6 - 05 Jul 2015, GeorgeClark
The copyright of the content on this website is held by the contributing authors, except where stated elsewhere. See Copyright Statement. Creative Commons License    Legal Imprint    Privacy Policy