Correction. serverUrl.getPort() returns -1 not serverUrl.getHost()

It might be an easy fix if standard http/https ports are assumed. However, for custom https ports, I can not think of any simple solution other than adding an attribute in the pages.xml to support custom http/https ports.

Unfortunately, this might not be the end of the story. There is a potential problem with determining the scheme using the request url as it is done in Pages.getRequestScheme(). That is, in a proxied environment, determining the scheme is a bit tricky. For example, a web server might communicate https with the browser but http with the app server, leaving the app server with no knowledge of https. Although, I have seen some web servers passing custom header values as hints to the app server.

What might work best for everybody is that if Seam can provide a pluggable implementation for the encodeScheme() functionality. This way, people with configurations similar to the ones mentioned above can write their own implementation of the encodeScheme() while everybody else can simply use the default Seam implementation.

Thank you so much for a quick response.

There is, however, one thing that is still a mystery to me. In the new code:

URL serverUrl = new URL(requestUrl);
StringBuilder sb = new StringBuilder();
sb.append(scheme);
sb.append("://");
sb.append(serverUrl.getHost());
if (serverUrl.getPort() != -1)
{
 sb.append(":");
 sb.append(serverUrl.getPort());
}

Although I am happy with the simple fix you added, I think it might help others if the http/https custom ports were to work also.

A JIRA issue created as you requestd:
http://jira.jboss.org/jira/browse/JBSEAM-1024