Merge pull request #3195 from strongloop/backport/fix-hash-path-in-redirect

Fix creation of verification links

Author: bajtos

common/models/user.js

@@ -13,6 +13,7 @@ var isEmail = require('isemail');
 var loopback = require('../../lib/loopback');
 var utils = require('../../lib/utils');
 var path = require('path');
+var qs = require('querystring');
 var SALT_WORK_FACTOR = 10;
 var crypto = require('crypto');
 var MAX_PASSWORD_LENGTH = 72;
@@ -428,10 +429,10 @@ module.exports = function(User) {
       options.host +
       displayPort +
       urlPath +
-      '?uid=' +
-      options.user[pkName] +
-      '&redirect=' +
-      options.redirect;
+      '?' + qs.stringify({
+        uid: options.user[pkName],
+        redirect: options.redirect,
+      });
 
     options.templateFn = options.templateFn || createVerificationEmailBody;
 

test/user.test.js

@@ -7,6 +7,7 @@ require('./support');
 var loopback = require('../');
 var User, AccessToken;
 var async = require('async');
+var url = require('url');
 
 describe('User', function() {
   this.timeout(10000);
@@ -1700,6 +1701,29 @@ describe('User', function() {
             expect(result.email).to.not.have.property('template');
           });
       });
+
+      it('allows hash fragment in redirectUrl', function() {
+        return User.create({email: '[email protected]', password: 'pass'})
+          .then(function(user) {
+            var actualVerifyHref;
+            return user.verify({
+              type: 'email',
+              to: user.email,
+              from: '[email protected]',
+              redirect: '#/some-path?a=1&b=2',
+              templateFn: function(options, cb) {
+                actualVerifyHref = options.verifyHref;
+                cb(null, 'dummy body');
+              },
+            })
+            .then(function() { return actualVerifyHref; });
+          })
+          .then(function(verifyHref) {
+            var parsed = url.parse(verifyHref, true);
+            expect(parsed.query.redirect, 'redirect query')
+              .to.equal('#/some-path?a=1&b=2');
+          });
+      });
     });
 
     describe('User.confirm(options, fn)', function() {