← Back to team overview

mahara-contributors team mailing list archive

[Bug 1714082] A change has been merged

 

Reviewed:  https://reviews.mahara.org/10037
Committed: https://git.mahara.org/mahara/mahara/commit/52ce6a21021dafc0a815621e7cbb2bbb81ef22a5
Submitter: Robert Lyon (robertl@xxxxxxxxxxxxxxx)
Branch:    master

commit 52ce6a21021dafc0a815621e7cbb2bbb81ef22a5
Author: Doris Tam <doristam@xxxxxxxxxxxxxxx>
Date:   Wed May 22 12:48:47 2019 +1200

Bug 1714082: prevent users from emails being case sensitive.

All emails are checked case-insensitively, but are kept in the db in
the way user has typed
- institution regisistrations are pending and don't appear in usr,
artefact, or artefact_internal_profile_email, but in usr_registration
so added additonal check
- placed helper function inside user.php
- putting in a special character before the @ will give single error
- putting in a special char after the @ will flag a spam checker and out
two errors

behatnotneeded

Change-Id: Iac1b740bc34878978ccf207d241f2399d7641040

-- 
You received this bug notification because you are a member of Mahara
Contributors, which is subscribed to Mahara.
Matching subscriptions: Subscription for all Mahara Contributors -- please ask on #mahara-dev or mahara.org forum before editing or unsubscribing it!
https://bugs.launchpad.net/bugs/1714082

Title:
  The user's primary email address is not checked in a case insensitive
  way

Status in Mahara:
  Fix Committed

Bug description:
  Currently you can create 2 users, via the Admin -> Users -> Add user
  screen, with this:

  username: Test one   email: UserA@xxxxxxxxxxx
  username: Test two   email: usera@xxxxxxxxxxx

  but not another user with

  username: Test three   email: usera@xxxxxxxxxxx

  As it complains the email address 'usera@xxxxxxxxxxx' is used

  However, with email addresses the domain part is case insensitive and
  only the username part is case sensitive - yet saying that most email
  servers treat the username part also as case insensitive to avoid
  email delivery confusion - see https://www.lifewire.com/are-email-
  addresses-case-sensitive-1171111

  To also avoid confusion within Mahara we should limit the primary
  email address to be case insensitive unique.

To manage notifications about this bug go to:
https://bugs.launchpad.net/mahara/+bug/1714082/+subscriptions


References