Hello,
 
There are two more bugs (report 741 and 744) that I would like to be fixed in version 3.2.2
Does anyone work on these bugs? If no one works on them I could try to fix them this weekend.
Can you please let me know.
 
Thank you very much,
Dimitris Dinodimos
 
Index: HttpRequestAdapter.java
===================================================================
RCS file: 
/home/cvspublic/jakarta-tomcat/src/share/org/apache/tomcat/service/http/Attic/HttpRequestAdapter.java,v
retrieving revision 1.16.2.5
diff -u -r1.16.2.5 HttpRequestAdapter.java
--- HttpRequestAdapter.java     2001/01/12 04:27:04     1.16.2.5
+++ HttpRequestAdapter.java     2001/01/13 10:11:09
@@ -247,21 +247,23 @@
     }
 
     public int getServerPort() {
-        if(serverPort == -1) {
-            //XXX Which is the default in case we can not get the port
-            //from the request??
-            serverPort=socket.getLocalPort();
-            String hostHeader = this.getHeader("host");
-            if (hostHeader != null) {
-                int i = hostHeader.indexOf(':');
-                if (i > -1) {
-                    hostHeader = hostHeader.substring(i+1);
-                    try{
-                        serverPort=Integer.parseInt(hostHeader);
-                    }catch(NumberFormatException pex){
-                    }
-                }else serverPort=80;
-            }
+        if (serverPort != -1)
+            return serverPort;
+        String hostHeader = this.getHeader("host");
+        if (hostHeader == null)
+            serverPort = socket.getLocalPort();
+        else {
+            int i = hostHeader.indexOf(':');
+            if (i > -1) {
+                hostHeader = hostHeader.substring(i+1);
+                try {
+                    serverPort = Integer.parseInt(hostHeader);
+                } catch (NumberFormatException pex) {
+                }
+            } else if (isSecure())
+                serverPort = 443;
+            else
+                serverPort = 80;
         }
         return serverPort;
     }

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, email: [EMAIL PROTECTED]

Reply via email to