Fix for JERSEY-2750: @RolesAllowed does not work in JettyHttpContainer

japod · japod · commit 99cafb39a980 · 2015-01-09T11:33:42.000+01:00
Change-Id: I8b897cf27221b9a63b9f5f2e1e52f7aabedc692a
Signed-off-by: Jakub Podlesak &lt;jakub.podlesak@oracle.com&gt;
diff --git a/containers/jetty-http/src/main/java/org/glassfish/jersey/jetty/JettyHttpContainer.java b/containers/jetty-http/src/main/java/org/glassfish/jersey/jetty/JettyHttpContainer.java
@@ -1,7 +1,7 @@
 /*
  * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS HEADER.
  *
- * Copyright (c) 2013-2014 Oracle and/or its affiliates. All rights reserved.
+ * Copyright (c) 2013-2015 Oracle and/or its affiliates. All rights reserved.
  *
  * The contents of this file are subject to the terms of either the GNU
  * General Public License Version 2 only ("GPL") or the Common Development
@@ -211,7 +211,7 @@ private SecurityContext getSecurityContext(final Request request) {
 
             @Override
             public boolean isUserInRole(final String role) {
-                return false;
+                return request.isUserInRole(role);
             }
 
             @Override

Original file line number	Diff line number	Diff line change
`@@ -1,7 +1,7 @@`
`1`	`1`	`/*`
`2`	`2`	`* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS HEADER.`
`3`	`3`	`*`
`4`		`- * Copyright (c) 2013-2014 Oracle and/or its affiliates. All rights reserved.`
	`4`	`+ * Copyright (c) 2013-2015 Oracle and/or its affiliates. All rights reserved.`
`5`	`5`	`*`
`6`	`6`	`* The contents of this file are subject to the terms of either the GNU`
`7`	`7`	`* General Public License Version 2 only ("GPL") or the Common Development`
`@@ -211,7 +211,7 @@ private SecurityContext getSecurityContext(final Request request) {`
`211`	`211`
`212`	`212`	`@Override`
`213`	`213`	`public boolean isUserInRole(final String role) {`
`214`		`- return false;`
	`214`	`+ return request.isUserInRole(role);`
`215`	`215`	`}`
`216`	`216`
`217`	`217`	`@Override`