Merge pull request DuendeArchive#305 from chaffeqa/patch-1

brockallen · web-flow · commit f021fb0944c6 · 2017-09-23T13:30:28.000-04:00
signoutRedirect respect post_logout_redirect_uri
diff --git a/src/UserManager.js b/src/UserManager.js
@@ -255,8 +255,18 @@ export default class UserManager extends OidcClient {
             return user;
         });
     }
-    signoutRedirect(args) {
+    signoutRedirect(args = {}) {
         Log.debug("UserManager.signoutRedirect");
+        let postLogoutRedirectUri = args.post_logout_redirect_uri || this.settings.post_logout_redirect_uri;
+        if (postLogoutRedirectUri){
+            args.post_logout_redirect_uri = postLogoutRedirectUri;
+            // we're putting a dummy entry in here because we 
+            // need a unique id from the state for notification
+            // to the parent window, which is necessary if we
+            // plan to return back to the client after signout
+            // and so we can close the popup after signout
+            args.state = args.state || {};
+        }
         return this._signoutStart(args, this._redirectNavigator).then(()=>{
             Log.info("signoutRedirect successful");
         });
