Ticket #6396 (closed Bug: fixed)

Opened 5 years ago

Last modified 5 years ago

Substring matching causes users to be rejected on signup?

Reported by: geeklibrarian Owned by: nouri
Priority: critical Milestone: 2.5.4
Component: Users/Groups Keywords:
Cc:

Description

Control Panel / Users and Groups

Create a user:

Full name: Scott W
User name: scottw

(Fill out the rest as needed)
Click on Register

User is added.

Now create another user:

Full name: Scott A
User name: scott

(Fill out the rest as needed)
Click on Register

The following error message results:

The login name you selected is already in use or is not valid. Please choose another.

I've been able to duplicate with any new user name by dropping the last letter from an existing user name (e.g. user name: "abcde" can't be added because user name: "abcdef" exists)

This is in a new instance, plone 3 beta on ploneout/windows, svn up'd on apr 6.

Change History

comment:1 Changed 5 years ago by limi

  • Priority changed from major to critical

Looks like substring matching of some sort. Wichert?

comment:2 Changed 5 years ago by limi

  • Summary changed from username rejected when it shouldn't be (add user function) to Substring matching causes users to be rejected on signup?

comment:3 Changed 5 years ago by gotcha

Same bug exists in 2.5.2

/CMFPlone/RegistrationTool.py(227)isMemberIdAllowed()

results = pas.searchPrincipals(id=id)
if results:
   return 0

the call to searchPrincipals returns results in case of substrings

comment:4 Changed 5 years ago by laz

Will check this one asap.

Maybe related to some other bugs I fixed at Sorrento ...

comment:5 Changed 5 years ago by laz

not enough close related unfortunately, no time to fix this one :-(

comment:6 Changed 5 years ago by nouri

  • Owner changed from wichert to nouri
  • Status changed from new to assigned
  • Milestone changed from 3.0 to 2.5.x

Fixed in r15171 for Plone 3 -- leaving open for milestone 2.5.x

comment:7 Changed 5 years ago by nouri

  • Status changed from assigned to closed
  • Resolution set to fixed

(In [15175]) Merge r15170:15174 from trunk into 2.5 branch. This fixes #6396

Note: See TracTickets for help on using tickets.