fix(medusa): Normalizes email before saving customer (#1719)

integration-tests/api/__tests__/store/customer.js

@@ -58,6 +58,21 @@ describe("/store/customers", () => {
       expect(response.data.customer).not.toHaveProperty("password_hash")
     })
 
+    it("normalizes email", async () => {
+      const api = useApi()
+
+      const response = await api.post("/store/customers", {
+        first_name: "James",
+        last_name: "Bond",
+        email: "James@Bond.com",
+        password: "test",
+      })
+
+      expect(response.status).toEqual(200)
+      expect(response.data.customer).not.toHaveProperty("password_hash")
+      expect(response.data.customer.email).toEqual("james@bond.com")
+    })
+
     it("responds 422 on duplicate", async () => {
       const api = useApi()
 

packages/medusa/src/services/customer.ts

@@ -258,6 +258,7 @@ class CustomerService extends TransactionBaseService<CustomerService> {
         this.customerRepository_
       )
 
+      customer.email = customer.email.toLowerCase()
       const { email, password } = customer
 
       const existing = await this.retrieveByEmail(email).catch(() => undefined)
@@ -289,7 +290,7 @@ class CustomerService extends TransactionBaseService<CustomerService> {
           delete customer.password
         }
 
-        const created = await customerRepository.create(customer)
+        const created = customerRepository.create(customer)
         const result = await customerRepository.save(created)
         await this.eventBusService_
           .withTransaction(manager)