From cab1335371dd16d456f956107414def68712b48a Mon Sep 17 00:00:00 2001 From: Miguel Serrano Date: Wed, 3 Jun 2020 11:37:06 +0200 Subject: [PATCH] Removed email lookup on CE user activation (#2879) GitOrigin-RevId: 567936295ef3b043adc7bbd735970f5ca6d91b73 --- services/web/app/src/Features/User/UserPagesController.js | 2 +- services/web/test/unit/src/User/UserPagesControllerTests.js | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/services/web/app/src/Features/User/UserPagesController.js b/services/web/app/src/Features/User/UserPagesController.js index 16d92817ac..07c7d724f8 100644 --- a/services/web/app/src/Features/User/UserPagesController.js +++ b/services/web/app/src/Features/User/UserPagesController.js @@ -52,7 +52,7 @@ const UserPagesController = { // Already seen this user, so account must be activate // This lets users keep clicking the 'activate' link in their email // as a way to log in which, if I know our users, they will. - res.redirect(`/login?email=${encodeURIComponent(user.email)}`) + res.redirect(`/login`) } else { req.session.doLoginAfterPasswordReset = true res.render('user/activate', { diff --git a/services/web/test/unit/src/User/UserPagesControllerTests.js b/services/web/test/unit/src/User/UserPagesControllerTests.js index d44477584e..3fbf951a99 100644 --- a/services/web/test/unit/src/User/UserPagesControllerTests.js +++ b/services/web/test/unit/src/User/UserPagesControllerTests.js @@ -328,7 +328,7 @@ describe('UserPagesController', function() { this.user.loginCount = 1 this.res.redirect = url => { this.UserGetter.getUser.calledWith(this.user_id).should.equal(true) - url.should.equal(`/login?email=${encodeURIComponent(this.user.email)}`) + url.should.equal('/login') return done() } return this.UserPagesController.activateAccountPage(this.req, this.res)