cvs commit: xml-axis/java/src/org/apache/axis/transport/http AxisServlet.java

[butek]

butek       02/03/21 08:19:09

  Modified:    java/src/org/apache/axis/transport/http AxisServlet.java
  Log:
  I backed out my 'fix' to get rid of a deprecation warning.  I also added a
  comment to make sure no-one else does what I just did.
  
  Revision  Changes    Path
  1.90      +7 -1      
xml-axis/java/src/org/apache/axis/transport/http/AxisServlet.java
  
  Index: AxisServlet.java
  ===================================================================
  RCS file: 
/home/cvs/xml-axis/java/src/org/apache/axis/transport/http/AxisServlet.java,v
  retrieving revision 1.89
  retrieving revision 1.90
  diff -u -r1.89 -r1.90
  --- AxisServlet.java  21 Mar 2002 14:13:17 -0000      1.89
  +++ AxisServlet.java  21 Mar 2002 16:19:09 -0000      1.90
  @@ -80,6 +80,7 @@
   import javax.servlet.http.HttpServletRequest;
   import javax.servlet.http.HttpServletResponse;
   import javax.servlet.http.HttpSession;
  +import javax.servlet.http.HttpUtils;
   import java.io.ByteArrayInputStream;
   import java.io.File;
   import java.io.IOException;
  @@ -255,7 +256,12 @@
   
               try {
   
  -                String url = req.getRequestURL().toString();
  +                // NOTE:  HttpUtils.getRequestURL has been deprecated.  This line 
SHOULD
  +                // be:  String url = req.getRequestURL().toString()
  +                // HOWEVER!!!!  DON'T REPLACE IT!  There's a bug in 
req.getRequestURL
  +                // that is not in HttpUtils.getRequestURL.  req.getRequestURL 
returns
  +                // "localhost" in the remote scenario rather than the actual host 
name.
  +                String url = HttpUtils.getRequestURL(req).toString();
   
                   msgContext.setProperty(MessageContext.TRANS_URL, url);