Figured out the issue of content-length being wrong when filter is in…

…voked

more than once in a request.

Doc/modpython4.tex

@@ -187,6 +187,9 @@ \section{Overview of a Filter Handler\label{pyapi-filter}}
 writing and reading should be done via the filter's object read and
 write methods.
 
+Filters need to be closed when a read operation returns None 
+(indicating End-Of-Stream).
+
 Similarly to a phase handler, a filter handler must return one of the
 status return codes.
 
@@ -210,7 +213,14 @@ \section{Overview of a Filter Handler\label{pyapi-filter}}
 
 def outputfilter(filter):
 
-    filter.write(filter.read().upper())
+    s = filter.read()
+    while s:
+        filter.write(s.upper())
+        s = filter.read()
+
+    if s is None:
+        filter.close()
+
     return apache.OK
 
 \end{verbatim}

lib/python/mod_python/apache.py

@@ -224,11 +224,11 @@ def FilterDispatch(self, filter):
                 else:
                     result = object(filter)
 
-                # always close the filter
-                filter.close()
+                # always flush the filter
+                filter.flush()
 
                 assert (type(result) == type(int())), \
-                       "Filter '%s' returned invalid return code." % filter.handler
+                       "Filter '%s' returned invalid return code: %s" % (filter.handler, `result`)
 
         except SERVER_RETURN, value:
             # SERVER_RETURN indicates a non-local abort from below

src/filterobject.c

@@ -57,7 +57,7 @@
  *
  * filterobject.c 
  *
- * $Id: filterobject.c,v 1.12 2002/09/16 21:33:39 grisha Exp $
+ * $Id: filterobject.c,v 1.13 2002/09/17 03:37:23 grisha Exp $
  *
  * See accompanying documentation and source code comments 
  * for details.
@@ -170,10 +170,12 @@ static PyObject *_filter_read(filterobject *self, PyObject *args, int readline)
 
     b = APR_BRIGADE_FIRST(self->bb_in);
 
-    /* reached eos on previous invocation? */
-    if (APR_BUCKET_IS_EOS(b) || b == APR_BRIGADE_SENTINEL(self->bb_in)) { 
-        if (b != APR_BRIGADE_SENTINEL(self->bb_in))
-            apr_bucket_delete(b);
+    if (b == APR_BRIGADE_SENTINEL(self->bb_in))
+        return PyString_FromString("");
+
+    /* reached eos ? */
+    if (APR_BUCKET_IS_EOS(b)) {
+        apr_bucket_delete(b);
         Py_INCREF(Py_None);
         return Py_None;
     }