From adc532cf742d4b76a0523dfb32c19d8d46fc76a8 Mon Sep 17 00:00:00 2001
From: Luca <perret.luca@gmail.com>
Date: Tue, 23 Jan 2018 13:35:51 +0100
Subject: [PATCH] feat(plugin-users-permissions): lowercase email

---
 .../controllers/Auth.js                            | 14 +++++++++-----
 1 file changed, 9 insertions(+), 5 deletions(-)

diff --git a/packages/strapi-plugin-users-permissions/controllers/Auth.js b/packages/strapi-plugin-users-permissions/controllers/Auth.js
index fe14f428da..8b6d14abbf 100644
--- a/packages/strapi-plugin-users-permissions/controllers/Auth.js
+++ b/packages/strapi-plugin-users-permissions/controllers/Auth.js
@@ -8,6 +8,7 @@
 
 const _ = require('lodash');
 const crypto = require('crypto');
+const emailRegExp = /^(([^<>()\[\]\\.,;:\s@"]+(\.[^<>()\[\]\\.,;:\s@"]+)*)|(".+"))@((\[[0-9]{1,3}\.[0-9]{1,3}\.[0-9]{1,3}\.[0-9]{1,3}\])|(([a-zA-Z\-0-9]+\.)+[a-zA-Z]{2,}))$/;
 
 module.exports = {
   callback: async (ctx) => {
@@ -29,14 +30,13 @@ module.exports = {
       const query = {};
 
       // Check if the provided identifier is an email or not.
-      const isEmail = /^(([^<>()\[\]\\.,;:\s@"]+(\.[^<>()\[\]\\.,;:\s@"]+)*)|(".+"))@((\[[0-9]{1,3}\.[0-9]{1,3}\.[0-9]{1,3}\.[0-9]{1,3}\])|(([a-zA-Z\-0-9]+\.)+[a-zA-Z]{2,}))$/.test(params.identifier);
+      const isEmail = emailRegExp.test(params.identifier);
 
       // Set the identifier to the appropriate query field.
-      const identifier = params.identifier.toLowerCase();
       if (isEmail) {
-        query.email = identifier;
+        query.email = params.identifier.toLowerCase();
       } else {
-        query.username = identifier;
+        query.username = params.identifier;
       }
 
       // Check if the user exists.
@@ -169,7 +169,11 @@ module.exports = {
       params.role = '1';
     }
 
-    params.identifier = params.identifier.toLowerCase();
+    // Check if the provided identifier is an email or not.
+    const isEmail = emailRegExp.test(params.identifier);
+    if (isEmail) {
+      params.identifier = params.identifier.toLowerCase();
+    }
     params.password = await strapi.plugins['users-permissions'].services.user.hashPassword(params);
 
     try {