minfrin in httpd

Propose, remove backported entry.

*) mod_http2: Fixes LimitRequestFields configuration handling to compensate for

browsers that send request headers as multiple ones to make best use of HTTP/2


Trunk version of patch:


svn merge -c 1879832 ^/httpd/httpd/trunk .

+1: icing, rpluem, minfrin

Vote, promote.

*) mod_http2: Avoid segfaults in case of handling certain responses for

already aborted connections.

Trunk version of patch:

Backport version for 2.4.x of patch:

+1: rpluem, icing, minfrin

Vote and promote.

mod_dav: Some DAV extensions, like CalDAV, specify both document

elements and property elements that need to be taken into account

when generating a property. The document element and property element

are made available in the dav_liveprop_elem structure under the

DAV_PROP_ELEMENT key in the resource pool.

mod_dav: Add utility functions dav_validate_root_ns(),

dav_find_child_ns(), dav_find_next_ns(), dav_find_attr_ns() and

dav_find_attr() so that other modules get to play too.


*) mod_watchdog: Switch to simpler logic to avoid the thread cleanup running

before the thread has started, avoiding mutex operations with undefined

behaviour. [Christophe Jaillet]

Vote and promote.

*) mod_http2: connection terminology renamed to master/secondary.

trunk patch:

2.4.x patch:

+1: icing, ylavic, minfrin

ylavic: nitpicking, mixed "H2_secondary_IN" and "H2_secondary_OUT" case to

register the filters, but not for adding them. IIRC filters names

are case insentive so shouldn't matter, just popped at my eyes..

icing: updated patch and added r1879156 to fix the eye bleed.

jailletc36: CHANGES could also be looked at if it makes sense to update the terminology

also here

*) core: Drop an invalid Last-Modified header value coming

from a (F)CGI script instead of replacing it with Unix epoch.

Warn the users about Last-Modified header value replacements

and violations of the RFC.

trunk patch:

2.4.x: trunk patches work, final view:

svn merge -c 1748379,1750747,1750749,1750953,1751138,1751139,1751139,1757818,1879253,r1879348 ^/httpd/httpd/trunk .

The code has been tested with a simple PHP script returning different Last-Modified

headers (GMT now, GMT now Europe/Paris, GMT tomorrow, GMT yesterday, PST now).

+1: elukey, jorton, jim

jorton: +1 though I'd say log at WARN or INFO for the APR_BAD_DATE case

rather than "silently" (at normal log-level) dropping the parsed header?

[also nit: wrapping a lone ap_log_rerror(,APLOG_X) call in

if (APLOGrX(..) is unnecessary/redundant]

  1. … 4 more files in changeset.
Update proposal.

Add logno to patch, add back CHANGES and ap_mmn.h.

    • ?
Bump the logno.

Update proposal.

Update proposal with a clean patch.

Use a brigade instead of direct reads, allow APR to handle MMAP.

Add docs to proposal.


Vote, vote.



Add MMN bump.

Add ETag patch.

    • ?
*) mod_proxy_fcgi: Don't unset when condition is false. PR64365

trunk patch:



2.4.x patch: svn merge -c 1877829,1877830 ^/httpd/httpd/trunk .

+1: covener, ylavic, rpluem

*) Replace apr_psprintf with apr_pstrcat where the format strings only

contain %s to improve efficiency. Leave out error messages as they

are not on a crtical code path and error message become less readable

when taking out the format specifiers.

trunk patch:

2.4.x patch: svn merge -c 1862270 ^/httpd/httpd/trunk .

+1: minfrin, rpluem, ylavic

Extend method_precondition hook to be called during the PROPFIND

and LABEL walkers, once for each walked resource.

Bump log numbers.