From e9dcfd04fd9a80660657c694c2c550ac96aa9eb2 Mon Sep 17 00:00:00 2001
From: roschaefer <git@roschaefer.de>
Date: Fri, 22 Nov 2019 15:04:06 +0100
Subject: [PATCH 1/3] Expose #2329

---
 webapp/components/LoginForm/LoginForm.spec.js | 73 +++++++++++++++++++
 1 file changed, 73 insertions(+)
 create mode 100644 webapp/components/LoginForm/LoginForm.spec.js

diff --git a/webapp/components/LoginForm/LoginForm.spec.js b/webapp/components/LoginForm/LoginForm.spec.js
new file mode 100644
index 000000000..b8697e771
--- /dev/null
+++ b/webapp/components/LoginForm/LoginForm.spec.js
@@ -0,0 +1,73 @@
+import LoginForm from './LoginForm.vue'
+import Styleguide from '@human-connection/styleguide'
+import Vuex from 'vuex'
+import { config, mount, createLocalVue } from '@vue/test-utils'
+
+const localVue = createLocalVue()
+localVue.use(Vuex)
+localVue.use(Styleguide)
+
+config.stubs['nuxt-link'] = '<span><slot /></span>'
+config.stubs['locale-switch'] = '<span><slot /></span>'
+config.stubs['client-only'] = '<span><slot /></span>'
+
+describe('LoginForm', () => {
+  let mocks
+  let propsData
+  let storeMocks
+
+  beforeEach(() => {
+    propsData = {}
+  })
+
+  describe('mount', () => {
+    const Wrapper = () => {
+      storeMocks = {
+        getters: {
+          'auth/pending': () => false,
+        },
+        actions: {
+          'auth/login': jest.fn(),
+        },
+      }
+      const store = new Vuex.Store(storeMocks)
+      mocks = {
+        $t: () => {},
+        $toast: {
+          success: jest.fn(),
+          error: jest.fn(),
+        },
+      }
+      return mount(LoginForm, { mocks, localVue, propsData, store })
+    }
+
+    describe('fill in email and password and submit', () => {
+      const fillIn = (wrapper, opts = {}) => {
+        const { email = 'email@example.org', password = '1234' } = opts
+        wrapper.find('input[name="email"]').setValue(email)
+        wrapper.find('input[name="password"]').setValue(password)
+        wrapper.find('form').trigger('submit')
+      }
+
+      it('dispatches login with form data', () => {
+        fillIn(Wrapper())
+        expect(storeMocks.actions['auth/login']).toHaveBeenCalledWith(
+          expect.any(Object),
+          { email: 'email@example.org', password: '1234' },
+          undefined,
+        )
+      })
+
+      describe('given email is a gmail address', () => {
+        it('removes dots, issue #2329', () => {
+          fillIn(Wrapper(), { email: 'example.user@gmail.com' })
+          expect(storeMocks.actions['auth/login']).toHaveBeenCalledWith(
+            expect.any(Object),
+            { email: 'exampleuser@gmail.com', password: '1234' },
+            undefined,
+          )
+        })
+      })
+    })
+  })
+})

From 145b6727b16ef1a7a9052286ea42873e61123785 Mon Sep 17 00:00:00 2001
From: roschaefer <git@roschaefer.de>
Date: Fri, 22 Nov 2019 15:08:25 +0100
Subject: [PATCH 2/3] Fix 2329 in the frontend

---
 webapp/components/LoginForm/LoginForm.vue | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/webapp/components/LoginForm/LoginForm.vue b/webapp/components/LoginForm/LoginForm.vue
index 91693ed4b..0d13772a9 100644
--- a/webapp/components/LoginForm/LoginForm.vue
+++ b/webapp/components/LoginForm/LoginForm.vue
@@ -73,6 +73,7 @@
 
 <script>
 import LocaleSwitch from '~/components/LocaleSwitch/LocaleSwitch.vue'
+import { normalizeEmail } from 'validator'
 
 export default {
   components: {
@@ -93,8 +94,10 @@ export default {
   },
   methods: {
     async onSubmit() {
+      let { email, password } = this.form
+      email = normalizeEmail(email)
       try {
-        await this.$store.dispatch('auth/login', { ...this.form })
+        await this.$store.dispatch('auth/login', { email, password })
         this.$toast.success(this.$t('login.success'))
         this.$emit('success')
       } catch (err) {

From ae360f2201c7dec466c2d16da0d1b6b8d23f264c Mon Sep 17 00:00:00 2001
From: roschaefer <git@roschaefer.de>
Date: Fri, 22 Nov 2019 15:24:27 +0100
Subject: [PATCH 3/3] fix #2329: Normalize email on login in the backend

---
 .../src/schema/resolvers/user_management.js   |  2 ++
 .../schema/resolvers/user_management.spec.js  | 24 +++++++++++++++++++
 webapp/components/LoginForm/LoginForm.spec.js | 11 ---------
 webapp/components/LoginForm/LoginForm.vue     |  4 +---
 4 files changed, 27 insertions(+), 14 deletions(-)

diff --git a/backend/src/schema/resolvers/user_management.js b/backend/src/schema/resolvers/user_management.js
index e1528cc9e..b7584dd68 100644
--- a/backend/src/schema/resolvers/user_management.js
+++ b/backend/src/schema/resolvers/user_management.js
@@ -2,6 +2,7 @@ import encode from '../../jwt/encode'
 import bcrypt from 'bcryptjs'
 import { AuthenticationError } from 'apollo-server'
 import { neode } from '../../bootstrap/neo4j'
+import { normalizeEmail } from 'validator'
 
 const instance = neode()
 
@@ -21,6 +22,7 @@ export default {
       // if (user && user.id) {
       //   throw new Error('Already logged in.')
       // }
+      email = normalizeEmail(email)
       const session = driver.session()
       const result = await session.run(
         `
diff --git a/backend/src/schema/resolvers/user_management.spec.js b/backend/src/schema/resolvers/user_management.spec.js
index 9d7dff2a3..df8454ebb 100644
--- a/backend/src/schema/resolvers/user_management.spec.js
+++ b/backend/src/schema/resolvers/user_management.spec.js
@@ -5,8 +5,10 @@ import { gql } from '../../helpers/jest'
 import { createTestClient } from 'apollo-server-testing'
 import createServer, { context } from '../../server'
 import encode from '../../jwt/encode'
+import { neode as getNeode } from '../../bootstrap/neo4j'
 
 const factory = Factory()
+const neode = getNeode()
 let query
 let mutate
 let variables
@@ -214,6 +216,28 @@ describe('login', () => {
           })
         })
       })
+
+      describe('normalization', () => {
+        describe('email address is a gmail address ', () => {
+          beforeEach(async () => {
+            const email = await neode.first('EmailAddress', { email: 'test@example.org' })
+            await email.update({ email: 'someuser@gmail.com' })
+          })
+
+          describe('supplied email contains dots', () => {
+            beforeEach(() => {
+              variables = { ...variables, email: 'some.user@gmail.com' }
+            })
+
+            it('normalizes email, issue #2329', async () => {
+              await respondsWith({
+                data: { login: expect.any(String) },
+                errors: undefined,
+              })
+            })
+          })
+        })
+      })
     })
 
     describe('with a valid email but incorrect password', () => {
diff --git a/webapp/components/LoginForm/LoginForm.spec.js b/webapp/components/LoginForm/LoginForm.spec.js
index b8697e771..b60680f37 100644
--- a/webapp/components/LoginForm/LoginForm.spec.js
+++ b/webapp/components/LoginForm/LoginForm.spec.js
@@ -57,17 +57,6 @@ describe('LoginForm', () => {
           undefined,
         )
       })
-
-      describe('given email is a gmail address', () => {
-        it('removes dots, issue #2329', () => {
-          fillIn(Wrapper(), { email: 'example.user@gmail.com' })
-          expect(storeMocks.actions['auth/login']).toHaveBeenCalledWith(
-            expect.any(Object),
-            { email: 'exampleuser@gmail.com', password: '1234' },
-            undefined,
-          )
-        })
-      })
     })
   })
 })
diff --git a/webapp/components/LoginForm/LoginForm.vue b/webapp/components/LoginForm/LoginForm.vue
index 0d13772a9..d61a5675a 100644
--- a/webapp/components/LoginForm/LoginForm.vue
+++ b/webapp/components/LoginForm/LoginForm.vue
@@ -73,7 +73,6 @@
 
 <script>
 import LocaleSwitch from '~/components/LocaleSwitch/LocaleSwitch.vue'
-import { normalizeEmail } from 'validator'
 
 export default {
   components: {
@@ -94,8 +93,7 @@ export default {
   },
   methods: {
     async onSubmit() {
-      let { email, password } = this.form
-      email = normalizeEmail(email)
+      const { email, password } = this.form
       try {
         await this.$store.dispatch('auth/login', { email, password })
         this.$toast.success(this.$t('login.success'))