fixed password reset by removing case sensitivity on email

Author: appinteractive

server/services/auth-management/auth-management.hooks.js

@@ -1,6 +1,6 @@
 const isEnabled = require('../../hooks/is-enabled');
 const { authenticate } = require('@feathersjs/authentication').hooks;
-const { iff } = require('feathers-hooks-common');
+const { iff, lowerCase } = require('feathers-hooks-common');
 
 const isAction = () => {
   let args = Array.from(arguments);
@@ -9,7 +9,9 @@ const isAction = () => {
 
 module.exports = {
   before: {
-    all: [],
+    all: [
+      lowerCase('value.email', 'value.username')
+    ],
     find: [],
     get: [],
     create: [

test/services/auth-management.test.js

@@ -55,6 +55,26 @@ describe('\'authManagement\' service', () => {
     });
     assert.ok(result, 'password can be reset');
   });
+
+  it('ignores uppercase email on password reset', async () => {
+    await service.create({
+      action: 'sendResetPwd',
+      value: {
+        email: user.email.toUpperCase()
+      }
+    });
+
+    const token = await getTokenFromMail();
+
+    const result = await service.create({
+      action: 'resetPwdLong',
+      value: {
+        token,
+        password: '123456'
+      }
+    });
+    assert.ok(result, 'password can be reset with upper case email');
+  });
 });
 
 const waitFor = (ms) => {
@@ -70,14 +90,18 @@ const getTokenFromMail = async () => {
   const mailDir = path.join(__dirname, '../../tmp/emails');
   // wait for the mail to be created first
   await waitFor(50);
-  const fileName = await fs.readdirSync(mailDir)[0];
+  const fileName = await fs.readdirSync(mailDir).pop();
   const mailFile = path.join(mailDir, fileName);
   const mailContent = await fs.readFileSync(mailFile, 'utf8');
   const $ = await cheerio.load(mailContent);
   let url;
   await $('a.btn-primary').each((i, el) => {
     url = el.attribs['href'];
   });
+
+  // delete mail
+  // await fs.unlinkSync(mailFile);
+
   const token = url.split('/').pop();
   return token;
 };