[portal-commits] JBoss Portal SVN: r8584 - in branches/JBoss_Portal_Branch_2_6: core-samples/src/main/org/jboss/portal/core/samples/basic and 1 other directory.

Author: thomas.heute(a)jboss.com Date: 2007-10-10 09:30:21 -0400 (Wed, 10 Oct 2007) New Revision: 8584 Modified: branches/JBoss_Portal_Branch_2_6/core-samples/src/main/org/jboss/portal/core/samples/basic/TestPortlet.java branches/JBoss_Portal_Branch_2_6/core/src/main/org/jboss/portal/core/impl/api/node/AbstractPortalNodeURL.java Log: JBPORTAL-1741: PortalURL security not honored With test links in the TestPortlet Modified: branches/JBoss_Portal_Branch_2_6/core/src/main/org/jboss/portal/core/impl/api/node/AbstractPortalNodeURL.java =================================================================== --- branches/JBoss_Portal_Branch_2_6/core/src/main/org/jboss/portal/core/impl/api/node/AbstractPortalNodeURL.java 2007-10-10 11:06:35 UTC (rev 8583) +++ branches/JBoss_Portal_Branch_2_6/core/src/main/org/jboss/portal/core/impl/api/node/AbstractPortalNodeURL.java 2007-10-10 13:30:21 UTC (rev 8584) @@ -53,6 +53,9 @@ /** . */ protected boolean relative; + /** . */ + protected URLContext urlContext; + public AbstractPortalNodeURL(PortalObjectId id, ControllerContext controllerContext) { this.id = id; @@ -74,11 +77,17 @@ public void setAuthenticated(Boolean authenticated) { this.wantAuthenticated = authenticated; + + // + this.urlContext = null; } public void setSecure(Boolean secure) { this.wantSecure = secure; + + // + this.urlContext = null; } public void setRelative(boolean relative) @@ -88,6 +97,41 @@ protected URLContext getURLContext() { - return controllerContext.getServerInvocation().getServerContext().getURLContext(); + if (urlContext == null) + { + URLContext tmp = controllerContext.getServerInvocation().getServerContext().getURLContext(); + + // + if (wantSecure != null) + { + if (wantSecure.booleanValue()) + { + tmp = tmp.asSecured(); + } + else + { + tmp = tmp.asNonSecured(); + } + } + + // + if (wantAuthenticated != null) + { + if (wantAuthenticated.booleanValue()) + { + tmp = tmp.asAuthenticated(); + } + else + { + tmp = tmp.asNonAuthenticated(); + } + } + + // + urlContext = tmp; + } + + // + return urlContext; } } Modified: branches/JBoss_Portal_Branch_2_6/core-samples/src/main/org/jboss/portal/core/samples/basic/TestPortlet.java =================================================================== --- branches/JBoss_Portal_Branch_2_6/core-samples/src/main/org/jboss/portal/core/samples/basic/TestPortlet.java 2007-10-10 11:06:35 UTC (rev 8583) +++ branches/JBoss_Portal_Branch_2_6/core-samples/src/main/org/jboss/portal/core/samples/basic/TestPortlet.java 2007-10-10 13:30:21 UTC (rev 8584) @@ -22,6 +22,7 @@ ******************************************************************************/ package org.jboss.portal.core.samples.basic; +import org.jboss.portal.api.node.PortalNodeURL; import org.jboss.portal.common.transaction.TransactionManagerProvider; import org.jboss.portal.core.CoreConstants; import org.jboss.portal.identity.User; @@ -147,7 +148,32 @@ writer.write("</div>"); writer.write("</p>"); + PortalNodeURL nodeURL = resp.createRenderURL(req.getPortalNode()); + String nothingURL = nodeURL.toString(); + nodeURL.setAuthenticated(Boolean.TRUE); + nodeURL.setSecure(Boolean.TRUE); + String authenticatedSecureURL = nodeURL.toString(); + nodeURL.setAuthenticated(Boolean.TRUE); + nodeURL.setSecure(Boolean.FALSE); + String authenticatedNotSecureURL = nodeURL.toString(); + nodeURL.setAuthenticated(Boolean.FALSE); + nodeURL.setSecure(Boolean.FALSE); + String notAuthenticatedNotSecureURL = nodeURL.toString(); + nodeURL.setAuthenticated(Boolean.FALSE); + nodeURL.setSecure(Boolean.TRUE); + String notAuthenticatedSecureURL = nodeURL.toString(); writer.write("<p>"); + writer.write("<div class=\"portlet-section-header\">Portal node URLs</div>"); + writer.write("<div class=\"portlet-section-body\">"); + writer.write("<div><a href=\"" + nothingURL + "\">Keeping context</a></div>"); + writer.write("<div><a href=\"" + authenticatedSecureURL + "\">Authenticated and Secured</a></div>"); + writer.write("<div><a href=\"" + authenticatedNotSecureURL + "\">Authenticated and Not Secured</a></div>"); + writer.write("<div><a href=\"" + notAuthenticatedNotSecureURL + "\">Not Authenticated and Not Secured</a></div>"); + writer.write("<div><a href=\"" + notAuthenticatedSecureURL + "\">Not Authenticated and Secured</a></div>"); + writer.write("</div>"); + writer.write("</p>"); + + writer.write("<p>"); writer.write("<div class=\"portlet-section-header\">Path</div>"); writer.write("<div class=\"portlet-section-body\">"); writer.write("<div>portal path info = " + req.getAttribute("PATH") + "</div>");