ci: fix upload-artifacts for linux dist build

Fixes [1]:

    Run actions/upload-artifact@v1
    Error: Path does not exist /home/runner/work/libevent/libevent/build
    Error: Exit code 1 returned from process: file name '/home/runner/runners/2.277.1/bin/Runner.PluginHost', arguments 'action "GitHub.Runner.Plugins.Artifact.PublishArtifact, Runner.Plugins"'.

  [1]: https://github.com/libevent/libevent/runs/2172680722?check_suite_focus=true

Merge branch 'http-fix-fd-leak'

* http-fix-fd-leak:
  http: fix fd leak on fd reset (by using bufferevent_replacefd())
  bufferevent: introduce bufferevent_replacefd() (like setfd() but also close fd)

Fixes: #1143

http: fix fd leak on fd reset (by using bufferevent_replacefd())

Fixes: afa66ea4 ("http: eliminate redundant bev fd manipulating and caching [WIP]")

bufferevent: introduce bufferevent_replacefd() (like setfd() but also close fd)

build: use AC_CHECK_LIB over AC_HAVE_LIBRARY

AC_HAVE_LIBRARY is deprecated, see
https://www.gnu.org/software/autoconf/manual/autoconf-2.70/html_node/Obsolete-Macros.html,
and has been prior to Autoconf 2.67, which is the minimum required by
the project. It's usage also causes warnings with newer versions of autoconf:
```bash
configure.ac:319: warning: The macro `AC_HAVE_LIBRARY' is obsolete.
configure.ac:319: You should run autoupdate.
```

`AC_HAVE_LIBRARY` was introduced in #969, although it's not clear why it
was decided to revert to using an obselete macro.

fix compile error in VS2012 in evutil_gettimeofday()

Move static variable definition before other statement, to prevent
compile error in VS2012.

Fixes: #1135

http: fix invalid unsigned arithmetic

Check return value of evbuffer_remove() in bufferevent_read()

The conflict cast convertion between the return value of
bufferevent_read() and evbuffer_remove(), int(-1)->size_t(An undefined
maximum)

Add test case of bufferevent_read() should return 0 in case of
evbuffer_remove() returns -1

Fixes: #1132

test/regress_http: cover reading of chunked payload by server

This is an example with transfer-encoding by client via bufferevent for
the #1131

evdns: add max-probe-timeout/probe-backoff-factor settings

I recently found that when the network status changed when calling
bufferevent_socket_connect_hostname (e.g. switching between several
WIFIs), all DNS servers would fail, and the timeout of probe would be
very long if there were many DNS requests. I want libevent to support
manual setting of MAX_PROBE_TIMEOUT and TIMEOUT_BACKOFF_FACTOR

So move hardcoded MAX_PROBE_TIMEOUT and TIMEOUT_BACKOFF_FACTOR into
struct, and allow changing them.

doc: remove dead link

The link is dead and an archived page cannot be found in the
web.archive.org.

cmake: do not link libevent with libevent_core

When add_event_library macro was called without the INNER_LIBRARIES parameter, it reused the
value set by a previous call, since the INNER_LIBRARIES variable was not reset.

Do not try to do SSL handshake if the connect() fails

This will avoid this icky error:

    $ https-client -4 -url https://127.1
    some request failed - no idea which one though!
    error:00000005:lib(0):func(0):DH lib

And instead will report only:

    $ https-client -4 -url https://127.1
    some request failed - no idea which one though!
    socket error = Connection refused (111)

Refs: #1115

Merge branch 'macos-brew-openssl-v2'

By Azat Khuzhin (2) and fanquake (1)
* macos-brew-openssl-v2:
  Add autodetection of openssl via brew into build matrix
  cmake: find openssl prefix via brew
  autotools: attempt to find OpenSSL via homebrew on macOS

Closes: #1050 (cherry picked one patch from it)

Add autodetection of openssl via brew into build matrix

cmake: find openssl prefix via brew

autotools: attempt to find OpenSSL via homebrew on macOS

When compiling for macOS, where users will likely have OpenSSL installed
via brew, rather than available on the system, use brew  --prefix to
figure out where OpenSSL is, and then augment the pkg-config path so
that libs are found.

Fix bufferevent_ssl_*flags() w/ disabled OpenSSL but enabled mbedTLS

Merge branch 'ssl-nagle'

* ssl-nagle:
  Introduce new BUFFEREVENT_SSL_BATCH_WRITE flag to avoid Nagle effect in SSL
  Introduce new API for flags of the SSL bufferevent

Introduce new BUFFEREVENT_SSL_BATCH_WRITE flag to avoid Nagle effect in SSL

Introduce new API for flags of the SSL bufferevent

Introduce more generic API (like for evbuffer):
- bufferevent_ssl_set_flags()
- bufferevent_ssl_clear_flags()
- bufferevent_ssl_get_flags()

And deprecate existing:
- bufferevent_openssl_get_allow_dirty_shutdown()
- bufferevent_openssl_set_allow_dirty_shutdown()
- bufferevent_mbedtls_get_allow_dirty_shutdown()
- bufferevent_mbedtls_set_allow_dirty_shutdown()

Merge branch 'ssl-fixes-after-mbedtls'

* ssl-fixes-after-mbedtls:
  Fix BEV_IS_SSL() macro
  Fix preprocessor condition for BEV_IS_SSL()
  Remove reduntant BEV_IS_MBEDTLS

Refs: #1028

Fix BEV_IS_SSL() macro

We cannot use the same trick with external declaration,
since there are copy of bufferevent_ops_ssl in each library:
- openssl
- mbedlts

However we can just compare the name of the bufferevent type for now.
(It is totally fine to use memcmp() here since it will be optimized by the compiler).

Fix preprocessor condition for BEV_IS_SSL()

Remove reduntant BEV_IS_MBEDTLS

http: eliminate redundant bev fd manipulating and caching [WIP]

At the very beginning we reset the bufferevent fd (if bev has it), which
is not a good idea, since if user passes bufferevent with existing fd he
has some intention.

So we need to:
- use BEV_OPT_CLOSE_ON_FREE for default bufferevent_socket_new() (to
  avoid manual shutdown/closee)
- drop getsockopt(SOL_SOCKET, SO_ERROR), since bufferevent already has
  evutil_socket_finished_connecting_()
- drop supperior bufferevent_setfd(bev, -1) in
  evhttp_connection_connect_()

Closes: #795
Refs: #875

Merge branch 'ipv6-tiny-improvements'

* ipv6-tiny-improvements:
  http: add EVHTTP_URI_HOST_STRIP_BRACKETS
  http-server: add -H option to change listen address
  https-client: add newline into error messages
  https-client: add -4/-6 switches

http: add EVHTTP_URI_HOST_STRIP_BRACKETS

Refs: #1115

http-server: add -H option to change listen address

https-client: add newline into error messages

https-client: add -4/-6 switches

becat: add timeout (client/server) and verbosity (event_enable_debug_logging())

Change log message in evutil_make_socket_nonblocking in win32

Co-authored-by: guoxiang2 <guoxiang2@yy.com>

Change prefix for man pages from "le_" to "libevent_"

As pointed by @kloczek:
  "With only two characters it would be harder to guess under what kind of name are libevent lvl 3 man pages.
   Longer name with bash/fix/zsh tab competition should not be a problem :)"

Refs: #1099

Add prefix (le_) for man pages

This will:
- avoid overlapping (since there is util.h.3 page)
- group everything under some prefix (man le_<TAB><TAB> will show everything)

Fixes: #1099

Drop DOXYGEN_MAN_LINKS unsupported cmake flag

MAN_LINKS is not a variable in Doxyfile anyway.

http-server: add cli argument for max body size

Remove dead code from bufferevent_socket_connect

Generate only real man pages with doxygen (MAN_LINKS=NO)

WIth MAN_LINKS=YES doxygen will generate ton's of man pages for each
field, function and so on, but it will not contain anything useful
except for link to another man page (header).

So to avoid man pages clashing, disable MAN_LINKS, and this will
generate man pages only for explicitly documented parts.

But even after this we have pretty non-unique man pages names:

    $ find install/ -name '*.3'
    install/share/man/man3/util.h.3
    ...

Looks like we should add prefix for each of them...
But let's see if there will be a problem.

Fixes: #1097

Remove man pages from repo (they can be generated via doxygen)

The problem with this man pages are:
- they are likely outdated
- they don't have install target
- and besides quality of man pages generated by doxygen not worse

sample: add https-client-mbedtls

test: fix windows error when mbedtls https

Merge #1087 - tests for https with mbedtls

* upstream/pr/1087:
  test: add https with mbedtls
  bufferevent_openssl: fix -Wcast-function-type for SSL_pending

Add build w/o any SSL support (i.e. w/o openssl and mbedtls)

This pure build w/o SSL has been removed in mbedtls PR - #1028

test: add https with mbedtls

bufferevent_openssl: fix -Wcast-function-type for SSL_pending

Introduced-in: #1028

http: const out address param to evcon_get_peer

build: fix building under windows with thread support

Fixes: 972289f3 ("Detect existence of pthread_mutexattr_setprotocol()")

Fix: long -> enum evhttp_cmd_type type

test/regress_dns: fix -Wmaybe-uninitialized

Was not noticed since it is reported only with optimization enabled, as
stated in gcc(1):

   The effectiveness of some warnings depends on optimizations also
   being enabled. For example -Wsuggest-final-types is more effective with
   link-time optimization and -Wmaybe-uninitialized does not warn at all
   unless optimization is enabled.

And interesting thing is that it is reported only for -O2, not for -O3,
that's why I did not catched it in both cmake env that I had:
- debug (it has -O0)
- release (it has -O3)

While autoconf has -O2.

Detect existence of pthread_mutexattr_setprotocol()

Fixes: #1084

build: add doxygen to all

Fixes: 7e6c5fb4 ("build: do not try install doxygen man pages if they were not requested")
Fixes: #1075
Fixes: #886

build: do not try install doxygen man pages if they were not requested

Fixes: #1075

Merge #1074

yuangongji:
  "Fix CI failures #1071
  - Install vcpkg through the source code to temporarily solve the windows CI problem.
  - Update cache version and replace setup-msys2."

* upstream/pr/1074:
  fix CI:Update cache version and replace setup-msys2
  fix CI: Install vcpkg through the source code to temporarily solve the windows CI problem

fix CI:Update cache version and replace setup-msys2

fix CI: Install vcpkg through the source code to temporarily solve the windows CI problem

Convert from WinCrypt to Windows BCrypt

Fixes: #1069

Handle return value from getrandom()

Remove reduntant variables in workflows with deploy

Fixes: bfbbc882 ("Change user.name/user.email to robot for deploy via github actions")

Change user.name/user.email to robot for deploy via github actions

test/dns: fix initialize_nameservers when there is ipv6 in /etc/resolv.conf

Fixes: #1060

windows socketpair tmpfile: use random prefix

fixes #1058

GetTempFileNameA() takes an optional prefix, and a "unique" long value
which can optionally be zero, which causes it to automatically
increment until a not-yet-existing filename is found.

When libevent creates many AF_UNIX socketpairs on windows,
it slows down dramatically, due to always using the same blank prefix,
and GetTempFileNameA() needing to iterate through all the existing
socketpair filenames. With a present and varying prefix, it will have
much less need to iterate.

It was also possible for a race with other processes also using
blank-prefix tmpfile names to result in both trying to start using
the same name at the same time (because libevent deletes the file
and then re-creates it as a unix socket), which should now be
much less likely.

Unfortuantely, "much" is just a factor of 32k, because the prefix
is only 3 characters, and windows filesystems are case-insensitive,
so doing better would require more sophisticated windows API usage
and charset trickyness.

fix: compat mbedtls < 2.4

Fix bufferevent_get_{openssl,mbedtls}_error()

The bufferevent_get_openssl_error() returns unsigned long, so returning
-1 on error in unclear. Let's use 0.

Fixes: #1028

ci/doxygen: install missing libmbedtls-dev

ci/coverage: add missing libmbedtls-dev

le-proxy: ignore SIGPIPE

Fixes: #1057

test/regress_buffer.c: fix -Wbad-function-cast (in some specific env)

Fixes: #1054

Merge branch 'mbedtls'

This patch set provides mbed TLS support.

The interface part (include/event2/bufferevent_ssl.h) LGTM, so this can
be safely merged (although there are some bits left).

Includes:
- bufferevent_mbedtls_*
- regress_mbedtls tests

Left:
- regress_http https_mbedtls support
- ChangeLog entry

* mbedtls:
  test: rename ssl/* -> openssl/*
  Join le_ssl_ops.post_init with le_ssl_ops.init
  Update LICENSE for ssl-client-mbedtls.c
  Merge ssl implementations (openssl and mbedtls)
  add mbedtls to CI
  fix build system and add test and cleanup code
  mbed TLS cmake support
  simple https client example using mbedtls
  mbedtls based SSL implementation

test: rename ssl/* -> openssl/*

Join le_ssl_ops.post_init with le_ssl_ops.init

Update LICENSE for ssl-client-mbedtls.c

Merge ssl implementations (openssl and mbedtls)

This patch splits common part out to avoid copy-paste from the
- bufferevent_openssl.c
- bufferevent_mbedtls.c

It uses VFS/bufferevent-like approach, i.e. structure of callbacks.

add mbedtls to CI

fix build system and add test and cleanup code

mbed TLS cmake support

FindMbedTLS.cmake is come from https://github.com/AVSystem/avs_commons/blob/master/cmake/FindMbedTLS.cmake, which is licensed under Apache 2.0

alternatives:
  https://github.com/curl/curl/blob/master/CMake/FindMbedTLS.cmake   without variable MBEDTLS_ROOT_DIR
  https://github.com/libgit2/libgit2/blob/master/cmake/Modules/FindmbedTLS.cmake  GPLv2 with a special Linking Exception

simple https client example using mbedtls

Based on mbedtls's source code programs/ssl/ssl_client1.c

mbedtls based SSL implementation

prototype is libevent-2.1.11-stable libevent_openssl.c

doc: fix typo in thread.h

s/evthred_use_pthreads_with_flags/evthread_use_pthreads_with_flags/

Merge #1048 -- append warning flags only if they are available (autotools)

* upstream/pr/1048:
  build: remove -Wstrict-aliasing as strict-aliasing is disabled
  build: move automake options into AM_INIT_AUTOMAKE call
  build: remove commented code from configure.ac
  build: use feature detection when adding compile flags
  build: add ax_check_compile_flag macro

build: remove -Wstrict-aliasing as strict-aliasing is disabled

Note that -Wstrict-aliasing is also include in -Wall.

build: move automake options into AM_INIT_AUTOMAKE call

build: remove commented code from configure.ac

build: use feature detection when adding compile flags

Rather than trying to detect (potentially very old) GCC versions, just
test whether the flag works with the compiler, and add it to CLFAGS if
so.

-Werror is used to convert unknown flag warnings into errors, and
prevent their addition to CLFAGS.

build: add ax_check_compile_flag macro

ci/linux: create dist artifact only if dist archive was built

Fixes: 512c88ce ("ci/linux: add dist check")

Merge branch 'fix-signal-leak'

Fixes main/fork under ASAN (LSAN if to be precise)

* fix-signal-leak:
  select: requires reinit (otherwise it leaks signal handlers)
  poll: requires reinit (otherwise it leaks signal handlers)

select: requires reinit (otherwise it leaks signal handlers)

Fixes: main/fork under SELECT with ASAN

poll: requires reinit (otherwise it leaks signal handlers)

Fixes: main/fork under POLL with ASAN

test: fix leak in dns/getaddrinfo_cancel_stress

Some requests may get response (evutil_addrinfo) from gaic_server_cb,
in case of cancel_event (10000ms) will not be fast enough.

test: fix UB in evbuffer/empty_reference_prepend_buffer

UBSAN reports:
  test/regress_buffer.c:2360:2: runtime error: null pointer passed as argument 1, which is declared to never be null
  /usr/include/string.h:140:33: note: nonnull attribute specified here

ci: set build type to debug with sanitizers

test: really disable bufferevent_pair_release_lock under ASAN (and fix gcc)

test-closed: fix leak

build: do not disable deprecation warnings on macOS

this was for using openssl-0.9.8 included in macOS 10.7 - 10.12,
but it is long since time you really should not use that openssl

Merge #1046 -- build: misc configure changes

* upstream/pr/1046:
  build: consolidate darwin compile flag additions in configure
  build: consistently use dnl for comments in configure.ac

Add abi-check report into artifacts

build: consolidate darwin compile flag additions in configure

This is mostly move-only, apart from some changes to comments.

build: consistently use dnl for comments in configure.ac

test: add getaddrinfo(AI_ADDRCONFIG) test (off by default)