remm        2005/04/28 05:30:39

  Modified:    http11/src/java/org/apache/coyote/http11
                        Http11AprProcessor.java
               catalina/src/share/org/apache/catalina/connector
                        Request.java
               catalina/src/share/org/apache/catalina/servlets
                        DefaultServlet.java
  Log:
  - Rename attributes as suggested by Bill.
  
  Revision  Changes    Path
  1.10      +6 -4      
jakarta-tomcat-connectors/http11/src/java/org/apache/coyote/http11/Http11AprProcessor.java
  
  Index: Http11AprProcessor.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-tomcat-connectors/http11/src/java/org/apache/coyote/http11/Http11AprProcessor.java,v
  retrieving revision 1.9
  retrieving revision 1.10
  diff -u -r1.9 -r1.10
  --- Http11AprProcessor.java   23 Apr 2005 17:33:40 -0000      1.9
  +++ Http11AprProcessor.java   28 Apr 2005 12:30:38 -0000      1.10
  @@ -1324,7 +1324,7 @@
   
           // Advertise sendfile support through a request attribute
           if (endpoint.getUseSendfile()) {
  -            request.setAttribute("sendfile.support", Boolean.TRUE);
  +            request.setAttribute("org.apache.tomcat.sendfile.support", 
Boolean.TRUE);
           }
           
       }
  @@ -1504,7 +1504,7 @@
           }
   
           // Sendfile support
  -        String fileName = (String) request.getAttribute("sendfile.filename");
  +        String fileName = (String) 
request.getAttribute("org.apache.tomcat.sendfile.filename");
           if (fileName != null) {
               // No entity body sent here
               outputBuffer.addActiveFilter
  @@ -1512,8 +1512,10 @@
               contentDelimitation = true;
               sendfileData = new AprEndpoint.SendfileData();
               sendfileData.fileName = fileName;
  -            sendfileData.start = ((Long) 
request.getAttribute("sendfile.start")).longValue();
  -            sendfileData.end = ((Long) 
request.getAttribute("sendfile.end")).longValue();
  +            sendfileData.start = 
  +                ((Long) 
request.getAttribute("org.apache.tomcat.sendfile.start")).longValue();
  +            sendfileData.end = 
  +                ((Long) 
request.getAttribute("org.apache.tomcat.sendfile.end")).longValue();
           }
           
           // Check for compression
  
  
  
  1.23      +2 -2      
jakarta-tomcat-catalina/catalina/src/share/org/apache/catalina/connector/Request.java
  
  Index: Request.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-tomcat-catalina/catalina/src/share/org/apache/catalina/connector/Request.java,v
  retrieving revision 1.22
  retrieving revision 1.23
  diff -u -r1.22 -r1.23
  --- Request.java      20 Apr 2005 16:13:23 -0000      1.22
  +++ Request.java      28 Apr 2005 12:30:38 -0000      1.23
  @@ -1367,7 +1367,7 @@
           }
   
           // Pass special attributes to the native layer
  -        if (name.startsWith("sendfile.")) {
  +        if (name.startsWith("org.apache.tomcat.")) {
               coyoteRequest.setAttribute(name, value);
           }
           
  
  
  
  1.34      +24 -10    
jakarta-tomcat-catalina/catalina/src/share/org/apache/catalina/servlets/DefaultServlet.java
  
  Index: DefaultServlet.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-tomcat-catalina/catalina/src/share/org/apache/catalina/servlets/DefaultServlet.java,v
  retrieving revision 1.33
  retrieving revision 1.34
  diff -u -r1.33 -r1.34
  --- DefaultServlet.java       27 Apr 2005 23:07:31 -0000      1.33
  +++ DefaultServlet.java       28 Apr 2005 12:30:38 -0000      1.34
  @@ -145,6 +145,13 @@
        */
       protected String fileEncoding = null;
       
  +    
  +    /**
  +     * Minimum size for sendfile usage in bytes.
  +     */
  +    protected int sendfileSize = 48 * 1024;
  +    
  +    
       // ----------------------------------------------------- Static 
Initializer
   
   
  @@ -235,6 +242,12 @@
               ;
           }
           try {
  +            value = getServletConfig().getInitParameter("sendfileSize");
  +            sendfileSize = Integer.parseInt(value) * 1024;
  +        } catch (Throwable t) {
  +            ;
  +        }
  +        try {
               value = getServletConfig().getInitParameter("fileEncoding");
               fileEncoding = value;
           } catch (Throwable t) {
  @@ -1488,21 +1501,22 @@
                                     HttpServletResponse response,
                                     CacheEntry entry,
                                     long length, Range range) {
  -        if ((entry.resource != null) 
  -            && ((length > 80 * 1024) || (entry.resource.getContent() == 
null))
  +        if ((sendfileSize > 0)
  +            && (entry.resource != null)
  +            && ((length > sendfileSize) || (entry.resource.getContent() == 
null))
               && (entry.attributes.getCanonicalPath() != null)
  -            && (Boolean.TRUE == request.getAttribute("sendfile.support"))
  +            && (Boolean.TRUE == 
request.getAttribute("org.apache.tomcat.sendfile.support"))
               && 
(request.getClass().getName().equals("org.apache.catalina.connector.RequestFacade"))
               && 
(response.getClass().getName().equals("org.apache.catalina.connector.ResponseFacade")))
 {
  -            request.setAttribute("sendfile.filename", 
entry.attributes.getCanonicalPath());
  +            request.setAttribute("org.apache.tomcat.sendfile.filename", 
entry.attributes.getCanonicalPath());
               if (range == null) {
  -                request.setAttribute("sendfile.start", new Long(0L));
  -                request.setAttribute("sendfile.end", new Long(length));
  +                request.setAttribute("org.apache.tomcat.sendfile.start", new 
Long(0L));
  +                request.setAttribute("org.apache.tomcat.sendfile.end", new 
Long(length));
               } else {
  -                request.setAttribute("sendfile.start", new 
Long(range.start));
  -                request.setAttribute("sendfile.end", new Long(range.end + 
1));
  +                request.setAttribute("org.apache.tomcat.sendfile.start", new 
Long(range.start));
  +                request.setAttribute("org.apache.tomcat.sendfile.end", new 
Long(range.end + 1));
               }
  -            request.setAttribute("sendfile.token", this);
  +            request.setAttribute("org.apache.tomcat.sendfile.token", this);
               return true;
           } else {
               return false;
  
  
  

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

Reply via email to