Repository: cxf Updated Branches: refs/heads/master 85d44cf51 -> a91805329
Another minor update to JAXRS SecurityContextImpl Project: http://git-wip-us.apache.org/repos/asf/cxf/repo Commit: http://git-wip-us.apache.org/repos/asf/cxf/commit/a9180532 Tree: http://git-wip-us.apache.org/repos/asf/cxf/tree/a9180532 Diff: http://git-wip-us.apache.org/repos/asf/cxf/diff/a9180532 Branch: refs/heads/master Commit: a91805329fc5de8bc194a86f24c70d93f4349084 Parents: 85d44cf Author: Sergey Beryozkin <[email protected]> Authored: Fri May 16 10:05:32 2014 +0100 Committer: Sergey Beryozkin <[email protected]> Committed: Fri May 16 10:05:32 2014 +0100 ---------------------------------------------------------------------- .../apache/cxf/jaxrs/impl/SecurityContextImpl.java | 15 +++++++++------ 1 file changed, 9 insertions(+), 6 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/cxf/blob/a9180532/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/impl/SecurityContextImpl.java ---------------------------------------------------------------------- diff --git a/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/impl/SecurityContextImpl.java b/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/impl/SecurityContextImpl.java index 9c7f24f..5127d7b 100644 --- a/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/impl/SecurityContextImpl.java +++ b/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/impl/SecurityContextImpl.java @@ -61,10 +61,7 @@ public class SecurityContextImpl implements SecurityContext { } public Principal getUserPrincipal() { - org.apache.cxf.security.SecurityContext sc = m.getContent(org.apache.cxf.security.SecurityContext.class); - if (sc == null) { - sc = m.get(org.apache.cxf.security.SecurityContext.class); - } + org.apache.cxf.security.SecurityContext sc = getInternalSecurityContext(); return sc == null ? null : sc.getUserPrincipal(); } @@ -75,9 +72,15 @@ public class SecurityContextImpl implements SecurityContext { } public boolean isUserInRole(String role) { - org.apache.cxf.security.SecurityContext sc = - m.get(org.apache.cxf.security.SecurityContext.class); + org.apache.cxf.security.SecurityContext sc = getInternalSecurityContext(); return sc == null ? false : sc.isUserInRole(role); } + private org.apache.cxf.security.SecurityContext getInternalSecurityContext() { + org.apache.cxf.security.SecurityContext sc = m.getContent(org.apache.cxf.security.SecurityContext.class); + if (sc == null) { + sc = m.get(org.apache.cxf.security.SecurityContext.class); + } + return sc; + } }
