Commit c962183b authored by Ruediger Pluem's avatar Ruediger Pluem
Browse files

Revert 1068310 as vetoed by Roy: 

This requirement will
be (or has already been) removed from httpbis because it hinders
extensibility and breaks content management systems, just as the
change below causes content-language to be broken.  That is why
we never implemented this requirement when it was added (without
consensus) to RFC 2616.



git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/branches/2.2.x@1068793 13f79535-47bb-0310-9956-ffa450edef68
parent c9725923

CHANGES

+0 −3
Changes for CHANGES: 0 added lines, 3 removed lines.
Original line number Diff line number Diff line
@@ -5,9 +5,6 @@ Changes with Apache 2.2.18 
     that we asked for to leak to the client should the 304 response be

     uncacheable. PR45341 [Graham Leggett]



  *) mod_dav: Send 501 error if unknown Content-* header is received for a PUT

     request (RFC 2616 9.6). PR 42978. [Stefan Fritsch]



  *) mod_dav: Send 400 error if malformed Content-Range header is received for

     a put request (RFC 2616 14.16). PR 49825. [Stefan Fritsch]

STATUS

+17 −0
Changes for STATUS: 17 added lines, 0 removed lines.
Original line number Diff line number Diff line
@@ -217,6 +217,23 @@ PATCHES PROPOSED TO BACKPORT FROM TRUNK: 


PATCHES/ISSUES THAT ARE STALLED



  * mod_dav: If an unknown Content-* header is received for a PUT request, we

    must not ignore it but reply with 501 per RFC 2616 9.6.

    PR: 42978

    Trunk version of patch:

      http://svn.apache.org/viewvc?rev=1026746&view=rev

    Backport version for 2.2.x of patch:

      Trunk version of patch works

    +1: rpluem, covener, poirier

    -1: roy:



This requirement will

be (or has already been) removed from httpbis because it hinders

extensibility and breaks content management systems, just as the

change below causes content-language to be broken.  That is why

we never implemented this requirement when it was added (without

consensus) to RFC 2616.



  * core: Support wildcards in both the directory and file components of

    the path specified by the Include directive.

    Trunk patch: http://svn.apache.org/viewvc?rev=909878&view=rev

modules/dav/main/mod_dav.c

+0 −32
Changes for modules/dav/main/mod_dav.c: 0 added lines, 32 removed lines.
Original line number Diff line number Diff line
@@ -812,30 +812,6 @@ static int dav_parse_range(request_rec *r, 
    return 1;

}



static const char *dav_validate_content_headers(request_rec *r)

{

    int i, prefix_len = strlen("content-");

    const apr_array_header_t *arr = apr_table_elts(r->headers_in);

    const apr_table_entry_t *elts = (const apr_table_entry_t *)arr->elts;



    for (i = 0; i < arr->nelts; ++i) {

         if (elts[i].key == NULL)

             continue;

         if (strncasecmp(elts[i].key, "content-", prefix_len) == 0

             && strcasecmp(elts[i].key + prefix_len, "length") != 0

             && strcasecmp(elts[i].key + prefix_len, "range") != 0

             /* Content-Location may be ignored per RFC 2616 14.14 */

             && strcasecmp(elts[i].key + prefix_len, "location") != 0

             && strcasecmp(elts[i].key + prefix_len, "type") != 0)

         {

             /* XXX: content-md5? content-language? content-encoding? */

             return apr_psprintf(r->pool, "Support for %s is not implemented.",

                                 ap_escape_html(r->pool, elts[i].key));

         }

    }

    return NULL;

}



/* handle the GET method */

static int dav_method_get(request_rec *r)

{
@@ -983,14 +959,6 @@ static int dav_method_put(request_rec *r) 
        mode = DAV_MODE_WRITE_TRUNC;

    }



    if ((body = dav_validate_content_headers(r)) != NULL) {

        /* RFC 2616 9.6: We must not ignore any Content-* headers we do not

         * understand.

         * XXX: Relax this for HTTP 1.0 requests?

         */

        return dav_error_response(r, HTTP_NOT_IMPLEMENTED, body);

    }



    /* make sure the resource can be modified (if versioning repository) */

    if ((err = dav_auto_checkout(r, resource,

                                 0 /* not parent_only */,