Linger on auth_callback page until login is complete

Verifying the auth token and obtaining user metadata involves some
async HTTP requests in the background.  If we remove the auth callback
information from the window location too soon, then the UserManager
will not complete the login process.  This currently generally works
because this process tends to complete before the /info and /tenant/info
calls to Zuul.  However, future changes to support a read-only
authentication requirement will need to alter this sequence.

This approach is more robust and easier to follow.  Essentially the
sequence is:
* Return from IdP to /auth_callback
* If /auth_callback is in our location, short-circuit normal rendering
  and render only the AuthCallbackPage
* When background processing is complete, onSignIn will be called and
  we will set the user.redirect property in redux.
* AuthCallbackPage has an effect callback on user.redirect which will
  cause it to perform the redirect away from auth_callback once
  signin is complete.

This process leaves the AuthCallbackPage on the screen a little longer,
so this change updates it to use the newer EmptyPage and Spinner elements
that we have been using elsewhere, so if users see it, it appears more
intentional.

Change-Id: I206c020626c7fd73e58efc29dd50376203679721

1 files changed, 3 insertions, 0 deletions

diff --git a/web/src/reducers/user.js b/web/src/reducers/user.js
index 1a36fdae1..215cbfb1a 100644
--- a/web/src/reducers/user.js
+++ b/web/src/reducers/user.js
@@ -29,6 +29,7 @@ export default (state = {
   scope: [],
   isAdmin: false,
   tenant: null,
+  redirect: null,
 }, action) => {
   switch (action.type) {
     case USER_LOGGED_IN: {
@@ -36,6 +37,7 @@ export default (state = {
         isFetching: false,
         data: action.user,
         token: action.token,
+        redirect: action.redirect,
         scope: [],
         isAdmin: false
       }
@@ -45,6 +47,7 @@ export default (state = {
         isFetching: false,
         data: null,
         token: null,
+        redirect: null,
         scope: [],
         isAdmin: false
       }