You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cxf.apache.org by co...@apache.org on 2017/04/12 11:14:31 UTC

cxf-fediz git commit: OIDC Logout URI should accept multiple Urls

Repository: cxf-fediz
Updated Branches:
  refs/heads/master 0a4667b4c -> f0259cf59


OIDC Logout URI should accept multiple Urls

Logout URI should accept multiple URLs separated
with a space.

Signed-off-by: Colm O hEigeartaigh <co...@apache.org>


Project: http://git-wip-us.apache.org/repos/asf/cxf-fediz/repo
Commit: http://git-wip-us.apache.org/repos/asf/cxf-fediz/commit/f0259cf5
Tree: http://git-wip-us.apache.org/repos/asf/cxf-fediz/tree/f0259cf5
Diff: http://git-wip-us.apache.org/repos/asf/cxf-fediz/diff/f0259cf5

Branch: refs/heads/master
Commit: f0259cf598f2b078634af44b37d9e301b487d29e
Parents: 0a4667b
Author: gonzalad <ad...@yahoo.fr>
Authored: Wed Apr 12 10:15:40 2017 +0200
Committer: Colm O hEigeartaigh <co...@apache.org>
Committed: Wed Apr 12 12:13:51 2017 +0100

----------------------------------------------------------------------
 .../fediz/service/oidc/clients/ClientRegistrationService.java | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/cxf-fediz/blob/f0259cf5/services/oidc/src/main/java/org/apache/cxf/fediz/service/oidc/clients/ClientRegistrationService.java
----------------------------------------------------------------------
diff --git a/services/oidc/src/main/java/org/apache/cxf/fediz/service/oidc/clients/ClientRegistrationService.java b/services/oidc/src/main/java/org/apache/cxf/fediz/service/oidc/clients/ClientRegistrationService.java
index a4859ca..b89f862 100644
--- a/services/oidc/src/main/java/org/apache/cxf/fediz/service/oidc/clients/ClientRegistrationService.java
+++ b/services/oidc/src/main/java/org/apache/cxf/fediz/service/oidc/clients/ClientRegistrationService.java
@@ -286,8 +286,11 @@ public class ClientRegistrationService {
             }
             // Client Logout URI
             if (!StringUtils.isEmpty(logoutURI)) {
-                if (!isValidURI(logoutURI, false)) {
-                    throwInvalidRegistrationException("An invalid logout URI was specified: " + logoutURI);
+                String[] logoutUris = logoutURI.split(" ");
+                for (String uri : logoutUris) {
+                    if (!isValidURI(uri, false)) {
+                        throwInvalidRegistrationException("An invalid logout URI was specified: " + uri);
+                    }
                 }
                 //TODO: replace this code with newClient.setLogoutUri() once it becomes available
                 newClient.getProperties().put("post_logout_redirect_uris", logoutURI);