feat(api): add custom redirect back

api-server/server/boot/authentication.js

@@ -54,8 +54,19 @@ module.exports = function enableAuthentication(app) {
   } else {
     api.get(
       '/signin',
+      (req, res, next) => {
+        if (req && req.query && req.query.returnTo) {
+          req.query.returnTo = `${homeLocation}/${req.query.returnTo}`;
+        }
+        return next();
+      },
       ifUserRedirect,
-      passport.authenticate('auth0-login', {})
+      (req, res, next) => {
+        const state = req.query.returnTo
+          ? Buffer.from(req.query.returnTo).toString('base64')
+          : null;
+        return passport.authenticate('auth0-login', { state })(req, res, next);
+      }
     );
 
     api.get(

api-server/server/component-passport.js

@@ -101,6 +101,10 @@ export const createPassportCallbackAuthenticator = (strategy, config) => (
   res,
   next
 ) => {
+  const returnTo =
+    req && req.query && req.query.state
+      ? Buffer.from(req.query.state, 'base64').toString('utf-8')
+      : `${homeLocation}/learn`;
   return passport.authenticate(
     strategy,
     { session: false },
@@ -112,7 +116,7 @@ export const createPassportCallbackAuthenticator = (strategy, config) => (
       if (!user || !userInfo) {
         return res.redirect('/signin');
       }
-      const redirect = `${homeLocation}/learn`;
+      const redirect = `${returnTo}`;
 
       const { accessToken } = userInfo;
       const { provider } = config;

api-server/server/passport-providers.js

@@ -34,6 +34,8 @@ export default {
     authPath: '/auth/auth0',
     callbackPath: '/auth/auth0/callback',
     useCustomCallback: true,
+    passReqToCallback: true,
+    state: false,
     successRedirect: successRedirect,
     failureRedirect: failureRedirect,
     scope: ['openid profile email'],

api-server/server/utils/middleware.js

@@ -56,10 +56,13 @@ export function ifNotVerifiedRedirectToUpdateEmail(req, res, next) {
 }
 
 export function ifUserRedirectTo(path = `${homeLocation}/`, status) {
-  status = status === 302 ? 302 : 301;
+  status = status === 301 ? 301 : 302;
   return (req, res, next) => {
     const { accessToken } = getAccessTokenFromRequest(req);
     if (req.user && accessToken) {
+      if (req.query && req.query.returnTo) {
+        return res.status(status).redirect(req.query.returnTo);
+      }
       return res.status(status).redirect(path);
     }
     if (req.user && !accessToken) {

client/src/client-only-routes/ShowSettings.js

@@ -162,7 +162,7 @@ export function ShowSettings(props) {
   }
 
   if (!showLoading && !isSignedIn) {
-    return navigate(`${apiLocation}/signin`);
+    return navigate(`${apiLocation}/signin?returnTo=settings`);
   }
 
   return (

client/src/client-only-routes/ShowSettings.test.js

@@ -11,7 +11,9 @@ describe('<ShowSettings />', () => {
     const shallow = new ShallowRenderer();
     shallow.render(<ShowSettings {...loggedOutProps} />);
     expect(navigate).toHaveBeenCalledTimes(1);
-    expect(navigate).toHaveBeenCalledWith(`${apiLocation}/signin`);
+    expect(navigate).toHaveBeenCalledWith(
+      `${apiLocation}/signin?returnTo=settings`
+    );
     expect(true).toBeTruthy();
   });
 });

client/src/pages/donate.js

@@ -91,7 +91,7 @@ export class DonatePage extends Component {
     }
 
     if (!showLoading && !isSignedIn) {
-      return navigate(`${apiLocation}/signin`);
+      return navigate(`${apiLocation}/signin?returnTo=donate`);
     }
 
     return (

client/src/pages/donate.test.js

@@ -11,7 +11,9 @@ describe('<ShowSettings />', () => {
     const shallow = new ShallowRenderer();
     shallow.render(<DonatePage {...loggedOutProps} />);
     expect(navigate).toHaveBeenCalledTimes(1);
-    expect(navigate).toHaveBeenCalledWith(`${apiLocation}/signin`);
+    expect(navigate).toHaveBeenCalledWith(
+      `${apiLocation}/signin?returnTo=donate`
+    );
     expect(true).toBeTruthy();
   });
 });

client/src/pages/portfolio.js

@@ -50,7 +50,7 @@ function ProfilePage(props) {
     return <Loader fullScreen={true} />;
   }
   if (!showLoading && !isSignedIn) {
-    return navigate(`${apiLocation}/signin`);
+    return navigate(`${apiLocation}/signin?returnTo=portfolio`);
   }
   const RedirecUser = createRedirect('/' + username);
   return <RedirecUser />;