HomePhabricator

Make auth.email-domains case-insensitive

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

Provenance
epriestleyAuthored on
epriestleyPushed on May 22 2014, 9:51 PM
Reviewer
btrahan
Differential Revision
D9261: Make auth.email-domains case-insensitive
Parents
rPc88385fa22a4: Tweak mobile action css, add to people
Branches
Unknown
Tags
Unknown
Tasks
T5143: Make "auth.email-domains" case-insensitive

Event Timeline