Andr� Malo wrote:

* Jeff Trawick wrote:


 +                rc = ap_get_brigade(r->input_filters, bb,
 AP_MODE_READBYTES,
 +                                    APR_BLOCK_READ, DAV_READ_BLOCKSIZE);
 +
 +                if (rc != APR_SUCCESS) {
 +                    err = dav_new_error(r->pool,
 HTTP_INTERNAL_SERVER_ERROR, 0,
 +                                        "Could not get next bucket
 brigade");
 +                    break;

don't we need to set keepalive flag to AP_CONN_CLOSE here and for any other error that causes us to stop reading the request body?


I thought, ap_status_drops_connection solves this automatically (for these
error codes)?

ah, I see that now... cool :)





Reply via email to