Homec4science

Make auth.email-domains case-insensitive

Authored by epriestley <git@epriestley.com> on May 22 2014, 23:51.

Description

Make auth.email-domains case-insensitive

Summary:
Fixes T5143. Currently, if your allowed domain is "example.com", we reject signups from "@Example.com".

Instead, lowercase both parts before performing the check.

Test Plan:

  • Before patch:
    • Set allowed domains to "yghe.net".
    • Tried "x@yghe.net", no error.
    • Tried "x@xxxy.net", error.
    • Tried "x@yghE.net", incorrectly results in an error.
  • After patch:
    • Set allowed domains to "yghe.net".
    • Tried "x@yghe.net", no error.
    • Tried "x@xxxy.net", error.
    • Tried "x@yghE.net", this correctly no longer produces an error.

Reviewers: btrahan

Reviewed By: btrahan

Subscribers: epriestley

Maniphest Tasks: T5143

Differential Revision: https://secure.phabricator.com/D9261

Details

Committed
epriestley <git@epriestley.com>May 22 2014, 23:51
Pushed
aubortJan 31 2017, 17:16
Parents
rPHc88385fa22a4: Tweak mobile action css, add to people
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rPHa76f61f7e183: Make auth.email-domains case-insensitive (authored by epriestley <git@epriestley.com>).May 22 2014, 23:51