You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by bi...@apache.org on 2006/01/28 23:39:46 UTC
svn commit: r373253 -
/tomcat/container/tc5.5.x/catalina/src/share/org/apache/catalina/realm/RealmBase.java
Author: billbarker
Date: Sat Jan 28 14:39:42 2006
New Revision: 373253
URL: http://svn.apache.org/viewcvs?rev=373253&view=rev
Log:
Don't append the port for an SSL redirect if it is the default port.
At the very least, appending the port is inefficient, and in some edge cases it can confuse MSIE no end.
Modified:
tomcat/container/tc5.5.x/catalina/src/share/org/apache/catalina/realm/RealmBase.java
Modified: tomcat/container/tc5.5.x/catalina/src/share/org/apache/catalina/realm/RealmBase.java
URL: http://svn.apache.org/viewcvs/tomcat/container/tc5.5.x/catalina/src/share/org/apache/catalina/realm/RealmBase.java?rev=373253&r1=373252&r2=373253&view=diff
==============================================================================
--- tomcat/container/tc5.5.x/catalina/src/share/org/apache/catalina/realm/RealmBase.java (original)
+++ tomcat/container/tc5.5.x/catalina/src/share/org/apache/catalina/realm/RealmBase.java Sat Jan 28 14:39:42 2006
@@ -876,9 +876,11 @@
String protocol = "https";
String host = request.getServerName();
// Protocol
- file.append(protocol).append("://");
+ file.append(protocol).append("://").append(host);
// Host with port
- file.append(host).append(":").append(redirectPort);
+ if(redirectPort != 443) {
+ file.append(":").append(redirectPort);
+ }
// URI
file.append(request.getRequestURI());
String requestedSessionId = request.getRequestedSessionId();
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org