Introduce StrictURI|UnsafeURI for RFC3986 enforcement

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1756959 13f79535-47bb-0310-9956-ffa450edef68

After lengthy investigation with covener's assistance, it seems we cannot
use a static table. We cannot change this to dynamic use of the local iconv
without build changes to avoid such use on cross-platform builds.

I'm satisfied if we trust iscntrl to at least catch all the most lethal
C0 Ctrls (we are promised it catches bad carriage control/line endings)
and leave this in the short term with an XXX to revisit at a future time.

The token stop never needed this table, because we can use the affirmative
list of token characters to define it.

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1756946 13f79535-47bb-0310-9956-ffa450edef68

Generally, the cart comes before the horse, this mirrors apr_lib.h

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1756937 13f79535-47bb-0310-9956-ffa450edef68

Resolve Netware (and other arch) build error for non-portable isascii()

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1756934 13f79535-47bb-0310-9956-ffa450edef68

Fix bogus K&R declarations

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1756927 13f79535-47bb-0310-9956-ffa450edef68

Resync docs

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1756863 13f79535-47bb-0310-9956-ffa450edef68

Fix as suggested by jailletc36, corresponding to all other comments on the topic

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1756862 13f79535-47bb-0310-9956-ffa450edef68

Fix some style (spaces only)

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1756853 13f79535-47bb-0310-9956-ffa450edef68

Be more consistent when we alocate this brigade.
This should save a few bytes in case of early exit and is, IMHO, cleaner.

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1756852 13f79535-47bb-0310-9956-ffa450edef68

A band-aid to resolve an immediate IBM MVS'ism

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1756849 13f79535-47bb-0310-9956-ffa450edef68

Replace tab by spaces to be consistent

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1756848 13f79535-47bb-0310-9956-ffa450edef68

Correct request header handling of whitespace with the new possible config of
HttpProtocolOptions Unsafe StrictWhitespace

I have elected not to preserve any significance to excess whitespace in the
now-deprecated obs-fold code path, that's certainly open for discussion.

This can be reviewed by tweaking t/conf/extra.conf to switch Strict to Unsafe.

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1756847 13f79535-47bb-0310-9956-ffa450edef68

Remove unused typedef uthn_dbd_conf

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1756846 13f79535-47bb-0310-9956-ffa450edef68

More values please

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1756845 13f79535-47bb-0310-9956-ffa450edef68

Remove unused typedef nvctx_t

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1756844 13f79535-47bb-0310-9956-ffa450edef68

Fix some missed "strcasecmp <--> ap_cstr_casecmp" conversions.

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1756843 13f79535-47bb-0310-9956-ffa450edef68

* server/protocol.c (read_request_line): Fix compiler warnings with
  GCC.

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1756824 13f79535-47bb-0310-9956-ffa450edef68

Drop redundant == --rrl_none evaluation, noted by rpluem

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1756823 13f79535-47bb-0310-9956-ffa450edef68

Drop unused, previously sscanf() target variables

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1756821 13f79535-47bb-0310-9956-ffa450edef68

Docs regen

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1756730 13f79535-47bb-0310-9956-ffa450edef68

Perform correct, strict parsing of the request line, handling the
http protocol tag, url and method appropriately, and attempting
to extract values even in the presence of unusual whitespace in
keeping with section 3.5, prior to responding with whatever
error reply is needed. Conforms to RFC7230 in all respects,
the section 3.5 optional behavior can be disabled by the user
with a new HttpProtocolOptions StrictWhitespace flag. In all
cases, the_request is regenerated from the parsed components
with exactly two space characters.

Shift sf's 'strict' method check from the Strict behavior because
it violates forward proxy logic, adding a new RegisteredMethods
flag, as it will certainly be useful to some.

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1756729 13f79535-47bb-0310-9956-ffa450edef68

Grab a pool of aplogno's

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1756727 13f79535-47bb-0310-9956-ffa450edef68

xforms

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1756707 13f79535-47bb-0310-9956-ffa450edef68

Improve highlight and formatting.

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1756706 13f79535-47bb-0310-9956-ffa450edef68

Renaming this directive to HttpProtocolOptions after discussion on dev@

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1756649 13f79535-47bb-0310-9956-ffa450edef68

mod_cache: follow up to r1756553: log the real/actual cached URI (debug).

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1756631 13f79535-47bb-0310-9956-ffa450edef68

mod_dav: follow up to r1746207:  fix typo (missing '/') for closing tag.

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1756611 13f79535-47bb-0310-9956-ffa450edef68

rebuild

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1756568 13f79535-47bb-0310-9956-ffa450edef68

Grammar tweaks, as per BZ59999

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1756567 13f79535-47bb-0310-9956-ffa450edef68

Correct AP_HTTP_CONFORMANCE_ flags with an ap_mmn.h bump

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1756555 13f79535-47bb-0310-9956-ffa450edef68

mod_cache: Use the actual URI path and query-string for identifying the
cached entity (key), such that rewrites are taken into account when
running afterwards (CacheQuickHandler off).  PR 21935.

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1756553 13f79535-47bb-0310-9956-ffa450edef68

mod_ssl: Fix quick renegotiation (OptRenegotiaton) with no intermediate
in the client certificate chain.  PR 55786.

This is done by handling an empty cert chain as no/NULL chain.

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1756542 13f79535-47bb-0310-9956-ffa450edef68

Regen docs

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1756541 13f79535-47bb-0310-9956-ffa450edef68

Rename the previously undocumented HTTPProtocol directive
to EnforceHTTPProtocol, and invert the default behavior
to strictly observe RFC 7230 unless otherwise configured.
And Document This.

The relaxation option is renamed 'Unsafe'. 'Strict' is no
longer case sensitive. 'min=0.9|1.0' is now the verbose
'Allow0.9' or 'Require1.0' case-insenstive grammer. The
exclusivity tests have been modified to detect conflicts.

The 'strict,log' option failed to enforce strict conformance,
and has been removed. Unsafe, informational logging is possible
in any loadable module, after the request data is unsafely
accepted.

This triggers a group of failures in t/apache/headers.t as
expected since those patterns violated RFC 7230 section 3.2.4.

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1756540 13f79535-47bb-0310-9956-ffa450edef68

Follow up to r1750392: reduce AH03408 level to INFO as suggested by wrowe/jim.

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1756531 13f79535-47bb-0310-9956-ffa450edef68

Follow up to r1750392: r1756186 should have also bumped MMN minor for
s/ap_proxy_check_backend/ap_proxy_check_connection/, and r1756328 really bump the MMN :p

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1756329 13f79535-47bb-0310-9956-ffa450edef68

Follow up to r1750392: r1756186 should have also bumped MMN minor for
s/ap_proxy_check_backend/ap_proxy_check_connection/.

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1756328 13f79535-47bb-0310-9956-ffa450edef68

Rebuild.

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1756274 13f79535-47bb-0310-9956-ffa450edef68

XML updates.

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1756273 13f79535-47bb-0310-9956-ffa450edef68

rebuild various recent changes

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1756216 13f79535-47bb-0310-9956-ffa450edef68

mod_proxy_fcgi: forcing proxy_conn->close here is useless, same is done
either above in ap_proxy_determine_connection(), or below in
ap_proxy_release_connection().

[Reverted by r1757097]

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1756187 13f79535-47bb-0310-9956-ffa450edef68

[mod_proxy_]http: follow up to r1750392.

Export [ap_]check_pipeline() and use it also for ap_proxy_check_connection(),
so that all the necessary checks on the connection are done before reusing it.

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1756186 13f79535-47bb-0310-9956-ffa450edef68

Clarify some z/OS mysteries via code comments.
Followup to r1421305.
PR 56210

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1756163 13f79535-47bb-0310-9956-ffa450edef68

Commiting to repor to not get later errors and continue the work elsewhere.

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1756150 13f79535-47bb-0310-9956-ffa450edef68

Revert r1756064 and r1756060 until fixed (tests framework passes).

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1756065 13f79535-47bb-0310-9956-ffa450edef68

http: follow up to r1750392.
Restore AP_MODE_READBYTES cleared inadvertently in r1756060.

[Reverted by r1756065]

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1756064 13f79535-47bb-0310-9956-ffa450edef68

[mod_proxy_]http: follow up to r1750392.
Export [ap_]check_pipeline() and use it also for ap_proxy_check_connection().

[Reverted by r1756065]

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1756060 13f79535-47bb-0310-9956-ffa450edef68

Fix spelling in comments and text files.
No functional change.
PR 59990

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1756038 13f79535-47bb-0310-9956-ffa450edef68

grammar++

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1755981 13f79535-47bb-0310-9956-ffa450edef68

rebuild. Remove weird copy/paste error.

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1755979 13f79535-47bb-0310-9956-ffa450edef68

indentation for better readability

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1755977 13f79535-47bb-0310-9956-ffa450edef68

rebuild various things

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1755974 13f79535-47bb-0310-9956-ffa450edef68

A little odd having that warning buried under the machine-readable
section.

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1755973 13f79535-47bb-0310-9956-ffa450edef68

Revert r1755930: actually what'd be read after AP_FCGI_END_REQUEST is padding,
not any potential/errorneous next response.

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1755931 13f79535-47bb-0310-9956-ffa450edef68

mod_{proxy,authnz}_fcgi: follow up to r1750392.
Don't try to read past AP_FCGI_END_REQUEST on the backend side, it would be
a response for the next request.

[Reverted by r1755931]

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1755930 13f79535-47bb-0310-9956-ffa450edef68

Silence more "defined but not used" compiler
warnings when building against OpenSSL 0.9.8a.

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1755881 13f79535-47bb-0310-9956-ffa450edef68

Silence "defined but not used" compiler warnings
when building against OpenSSL 0.9.8a.

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1755874 13f79535-47bb-0310-9956-ffa450edef68

OpenSSL 1.1.0 compat:
- move IDCONST macro outside of addition #if check.
  Otherwise we break compatibility with old
  OpenSSL 0.9.8 releases.

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1755856 13f79535-47bb-0310-9956-ffa450edef68

Follow on to r1755264, for the case of merged header length exceptions,
and ensure the field header name is truncated to a sane log width.

  Stop reflecting irrelevant data to the request error notes, particularly
  for abusive and malformed traffic the non-technical consumer of a user-agent
  has no control over.

  Simply take note where the administrator-configured limits have been exceeded,
  that administrator can find details in the error log if desired.

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1755744 13f79535-47bb-0310-9956-ffa450edef68

Remove calling ERR_remove_thread_state when
using OpenSSL 1.1.0.

This API is now a no-op in OpenSSL 1.1.0 and
deprecated.

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1755725 13f79535-47bb-0310-9956-ffa450edef68

Compatibility with OpenSSL 1.1.0 pre6.

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1755657 13f79535-47bb-0310-9956-ffa450edef68

Clarify phrasing, as per bz53152

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1755361 13f79535-47bb-0310-9956-ffa450edef68

Various typos, broken links, and obsolete modules.

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1755355 13f79535-47bb-0310-9956-ffa450edef68

fix link to mod_headers

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1755347 13f79535-47bb-0310-9956-ffa450edef68

rebuild

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1755345 13f79535-47bb-0310-9956-ffa450edef68

remove references to MPMs that no longer exist

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1755344 13f79535-47bb-0310-9956-ffa450edef68

Follow up to r1755264.
Don't crash when ap_rgetline() returns a NULL field on ENOSPC.

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1755343 13f79535-47bb-0310-9956-ffa450edef68

rebuild fixed links

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1755342 13f79535-47bb-0310-9956-ffa450edef68

Correct name of authnz_ldap module in French translation.

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1755340 13f79535-47bb-0310-9956-ffa450edef68

Remove reference to long-gone FAQ.

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1755338 13f79535-47bb-0310-9956-ffa450edef68

rebuild

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1755337 13f79535-47bb-0310-9956-ffa450edef68

Remove reference to long-gone FAQ.

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1755336 13f79535-47bb-0310-9956-ffa450edef68

fix link to mod_setenvif

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1755335 13f79535-47bb-0310-9956-ffa450edef68

rebuild

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1755331 13f79535-47bb-0310-9956-ffa450edef68

fix broken links

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1755330 13f79535-47bb-0310-9956-ffa450edef68

Update

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1755326 13f79535-47bb-0310-9956-ffa450edef68

We didn't change the MODULE_MAGIC_NUMBER_MAJOR

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1755325 13f79535-47bb-0310-9956-ffa450edef68

rebuild

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1755324 13f79535-47bb-0310-9956-ffa450edef68

mod_http2: support for intermediate responses

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1755323 13f79535-47bb-0310-9956-ffa450edef68

Update link to new location of Trawick's Exception Hook stuff.

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1755322 13f79535-47bb-0310-9956-ffa450edef68

Stop reflecting irrelevant data to the request error notes, particularly
for abusive and malformed traffic the non-technical consumer of a user-agent
has no control over.

Simply take note where the administrator-configured limits have been exceeded,
that administrator can find details in the error log if desired.

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1755264 13f79535-47bb-0310-9956-ffa450edef68

Treat an empty obs-fold line as abusive traffic

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1755263 13f79535-47bb-0310-9956-ffa450edef68

Strictly observe the one-SP requirement of every obs-fold line.

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1755236 13f79535-47bb-0310-9956-ffa450edef68

Now eliminate all intra-obs-fold excess whitespace, results condensed per spec.

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1755235 13f79535-47bb-0310-9956-ffa450edef68

An empty obs-fold line is a no-op.

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1755234 13f79535-47bb-0310-9956-ffa450edef68

As promised, reduce this logic by net 9 code lines, shifting the burden
of killing trailing whitespace to the purpose-agnostic read logic.

Whitespace before or after an obs-fold, and before or after a field value
have no semantic purpose at all. Because we are building a buffer for all
folded values, reducing the size of the newly allocated buffer is always
to our advantage.

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1755233 13f79535-47bb-0310-9956-ffa450edef68

Another comment for reviewers to follow

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1755126 13f79535-47bb-0310-9956-ffa450edef68

Testing len > 0 is redundant when *field is a "\0" and mismatches here,
folded flag was a no-op, unused once we added continue; logic.
Fix these as initially attempted in r1755114.

Add some comments to make this code slightly more legible to the reviewer.

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1755125 13f79535-47bb-0310-9956-ffa450edef68

Reformat for indentation following r1755123, Whitespace Only

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1755124 13f79535-47bb-0310-9956-ffa450edef68

Clean up an edge case where obs-fold continuation preceeds the first header,
as with r1755098, but this time ensure the previous header processing logic
ensures there was a previous header as identified by jchampion.

This patch restructures the loop for legibility with a loop continuation,
allowing us to flatten all of this hard-to-follow code. The subsequent
patch will be a whitespace-only change for formatting.

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1755123 13f79535-47bb-0310-9956-ffa450edef68

Revert to r1754577 based on last_field NULL on first-pass defect
noted by Jacob Champion, discarding my work of today.

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1755118 13f79535-47bb-0310-9956-ffa450edef68

C89 fix, caught by Jacob Champion

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1755116 13f79535-47bb-0310-9956-ffa450edef68

Improve comments for reviewers to understand this logic

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1755115 13f79535-47bb-0310-9956-ffa450edef68

Testing len > 0 is redundant when *field is a "\0" and mismatches here.

folded flag was a no-op, unused once we added continue; logic.

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1755114 13f79535-47bb-0310-9956-ffa450edef68

Whitespace-only followup after r1755098 logic patch

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1755099 13f79535-47bb-0310-9956-ffa450edef68

Clean up an edge case where obs-fold continuation preceeds the first header.

This patch restructures the loop for legibility with a loop continuation,
allowing us to flatten all of this hard-to-follow code. The subsequent
patch will be a whitespace-only change for formatting.

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1755098 13f79535-47bb-0310-9956-ffa450edef68

Revert r1754732,r1754734,r1754736 after a conversation on dev@ and IRC.
We agreed to consider r1752347 as bug fix and delay performance improvements
for later revisions.

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1754983 13f79535-47bb-0310-9956-ffa450edef68

mod_proxy_http: 204 or 304 cannot happen here since ap_is_HTTP_ERROR() above
is true.

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1754979 13f79535-47bb-0310-9956-ffa450edef68

mod_proxy_http: handle 401 error override in the same block as other errors
to simplify logic, no functional change.

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1754975 13f79535-47bb-0310-9956-ffa450edef68

mod_proxy_http: don't pretend we are sending a body before ProxyErrorOverride
takes place, the response it to be handled by ErrorDocument is this case.
Also it useless to take care of pre-read data since the body is discarded in
this case.

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1754974 13f79535-47bb-0310-9956-ffa450edef68