summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* docs/make: generate curl.1 from listed files onlybagder/mk-manpage-from-dpagesDaniel Stenberg2020-03-262-40/+39
| | | | | | | | | | | | Previously it rendered the page from files matching "*.d" in the correct directory, which worked fine in git builds when the files were added but made it easy to forget adding the files to the dist. Now, only man page sections listed in DPAGES in Makefile.inc will be used, thus "forcing" us to update this to get the man page right and get it included in the dist at the same time. Ref: #5146
* openssl: adapt to functions marked as deprecated since version 3Daniel Stenberg2020-03-263-0/+58
| | | | | | | | | | | | | | OpenSSL 3 deprecates SSL_CTX_load_verify_locations and the MD4, DES functions we use. Fix the MD4 and SSL_CTX_load_verify_locations warnings. In configure, detect OpenSSL v3 and if so, inhibit the deprecation warnings. OpenSSL v3 deprecates the DES functions we use for NTLM and until we rewrite the code to use non-deprecated functions we better ignore these warnings as they don't help us. Closes #5139
* dist: add mail-rcpt-allowfails.d to the tarballDaniel Stenberg2020-03-251-2/+3
| | | | | | | Reported-by: Maksim Stsepanenka Reviewed-by: Jat Satiro Closes #5146
* travis: update the ngtcp2 build to use the latest OpenSSL patchDaniel Stenberg2020-03-251-1/+1
| | | | ... which also makes it OpenSSL 1.1.1d based and not v3.
* CI: remove default Ubuntu build from GitHub ActionsMarc Hoersken2020-03-241-17/+0
| | | | | We are already running a very similar Ubuntu build on Travis CI. The macOS variant of this default build is kept on Github Actions.
* CI: bring GitHub Actions fuzzing job in line with macOS jobsMarc Hoersken2020-03-241-3/+7
| | | | Update YAML formatting, job naming and triggers.
* CI: migrate macOS jobs from Azure and Travis CI to GitHub ActionsMarc Hoersken2020-03-243-107/+88
| | | | | | | | | Reduce workload on Azure Pipelines and Travis CI while consolidating macOS jobs onto less utilized GitHub Actions. Reviewed-by: Daniel Stenberg Closes #5124
* config: remove all defines of HAVE_DES_HDaniel Stenberg2020-03-247-22/+2
| | | | | | As there's no code using it. Closes #5144
* copyright: fix out-of-date copyright ranges and missing headersDaniel Stenberg2020-03-24353-356/+2240
| | | | | | | | | Reported by the new script 'scripts/copyright.pl'. The script has a regex whitelist for the files that don't need copyright headers. Removed three (mostly usesless) README files from docs/ Closes #5141
* packages: add OS400/chkstrings.c to the distDaniel Stenberg2020-03-241-0/+1
| | | | | | Reported-by: Jon Rumsey Fixes #5142 Closes #5143
* nghttp2: 1.12.0 requiredClément Notin2020-03-231-1/+1
| | | | | | since nghttp2_session_set_local_window_size is needed Closes #5140
* RELEASE-NOTES: syncedDaniel Stenberg2020-03-231-6/+27
|
* OS400: Update strings for ccsid-ifierCalvin Buckley2020-03-232-6/+9
| | | | | | Fixes build. Closes #5132
* cirrus: make freebsd ignore the tests instead of skippingDaniel Stenberg2020-03-231-3/+3
| | | | | | To allow us to see in the CI logs how they actually behave Closes #5091
* cirrus: move the sanitizer build from freebsd 13 to freebsd 12Daniel Stenberg2020-03-231-2/+1
|
* Revert "cirrus-ci: disable the FreeBSD 13 builds"Daniel Stenberg2020-03-231-1/+1
| | | | This reverts commit 691b71be930f0e285c8f7a76efd56bbe0576cda6.
* getinfo: provide CURLINFO_HEADER_SIZE and CURLINFO_REQUEST_SIZE overrideDaniel Stenberg2020-03-222-1/+15
| | | | | | | To let debug-builds return fake values, like in test 970. Ref: #5131 Closes #5136
* test970: improve the testDaniel Stenberg2020-03-221-19/+7
| | | | | | | | | - send more data to make problems more obvious - don't start the data with minus, it makes diffs harder to read - skip the headers in the stdout comparison - save to a file name to also verify 'filename_effective' Ref: #5131
* CURLINFO_NUM_CONNECTS: improve accuracyDaniel Stenberg2020-03-221-1/+1
| | | | | | | | The counter was not bumped in all cases correctly. Reported-by: Marcel Raad Ref: #5131 Closes #5135
* TODO: Use "random" ports for the test serversDaniel Stenberg2020-03-221-0/+12
|
* lib/curl_setup: adjust the copyright year rangeDaniel Stenberg2020-03-211-1/+1
| | | | Follow-up from d820224b8
* curl_setup: define _WIN32_WINNT_[OS] symbolsJay Satiro2020-03-212-51/+130
| | | | | | | | | | | | | | | | .. because not all Windows build systems have those symbols, and even those that do may be missing newer symbols (eg the Windows 7 SDK does not define _WIN32_WINNT_WIN10). Those symbols are used in build-time logic to decide which API to use and prior to this change if the symbols were missing it would have resulted in deprecated API being used when more recent functions were available (eg GetVersionEx used instead of VerifyVersionInfo). Reported-by: FuccDucc@users.noreply.github.com Probably fixes https://github.com/curl/curl/issues/4995 Closes https://github.com/curl/curl/pull/5057
* curl-functions.m4: remove inappropriate AC_REQUIRERoss Burton2020-03-211-2/+0
| | | | | | | | | | | | | AC_REQUIRE means "if this macro hasn't been executed already, execute it". So in a wrapper around AC_RUN_IFELSE, AC_REQUIRE(AC_RUN_IFELSE) isn't correct at that will execute AC_RUN_IFELSE without any arguments. With autoconf 2.69 this is basically a no-op, but with autoconf 2.70, AC_RUN_IFELSE without a default value when cross-compiling is fatal. The result is that curl with autoconf 2.70 cannot cross-compile. Fixes https://github.com/curl/curl/issues/5126 Closes https://github.com/curl/curl/pull/5130
* ci/tests: fix Azure Pipelines not running Windows containersMarc Hoersken2020-03-201-0/+2
| | | | | | | | | | Workaround posted here: microsoft/azure-pipelines-agent#2864 Assisted-by: Simon Chalifoux Assisted-by: Tommy Petty Fixes #5117 Closes #5129
* tests: add test 430, 431 and 432 to verify the --config fixDaniel Stenberg2020-03-204-0/+298
| | | | Verify the fixes in 4e0b4fee4
* cmdline: fix handling of OperationConfig linked list (--next)Rici Lake2020-03-202-2/+4
| | | | | | | | | | Ensures that -K/--config inserts new items at the end of the list instead of overwriting the second item, and that after a -K/--config option has been parsed, the option parser's view of the current config is update. Fixes #5120 Closes #5123
* test2100: fix static port instead of dynamic value being usedMarc Hoersken2020-03-201-0/+0
|
* test970: fix static ip:port instead of dynamic values being usedMarc Hoersken2020-03-201-1/+1
|
* secure transport: remove the BACKEND define kludgeDaniel Stenberg2020-03-191-90/+101
| | | | Closes #5122
* mbedtls: remove the BACKEND define kludgeDaniel Stenberg2020-03-191-69/+76
|
* bearssl: remove the BACKEND define kludgeDaniel Stenberg2020-03-191-48/+52
|
* wolfssl: remove the BACKEND define kludgeDaniel Stenberg2020-03-191-59/+65
|
* nss: remove the BACKEND define kludgeDaniel Stenberg2020-03-191-51/+62
|
* gnutls: remove the BACKEND define kludgeDaniel Stenberg2020-03-191-43/+50
|
* openssl: remove the BACKEND define kludgeDaniel Stenberg2020-03-191-107/+113
| | | | | Use a proper variable instead to make it easier to use a debugger and read the code.
* tests: make Python-based servers compatible with Python 2 and 3Marc Hoersken2020-03-197-23/+58
| | | | | | | | | | | | | | | | Update smbserver.py and negtelnetserver.py to be compatible with Python 3 while staying backwards-compatible to support Python 2. Fix string encoding and handling of echoed and transferred data. Tested with both Python 2.7.17 and Python 3.7.7 Reported-by: Daniel Stenberg Assisted-by: Kamil Dudka Reviewed-by: Marcel Raad Fixes #5104 Closes #5110
* writeout_json: use curl_off_t printf() option for the time outputDaniel Stenberg2020-03-181-1/+2
| | | | | | Follow-up to: 04c03416e68fd635a15 Closes #5115
* RELEASE-NOTES: syncedDaniel Stenberg2020-03-181-0/+2
| | | | Uh, I missed this in 1a46b218db
* RELEASE-NOTES: syncedDaniel Stenberg2020-03-182-10/+45
| | | | ... and bumped curlver.h to 7.70.0
* http2: Fix erroneous debug message that h2 connection closedJay Satiro2020-03-182-8/+14
| | | | | | | | | | | | | Prior to this change in libcurl debug builds http2 stream closure was erroneously referred to as connection closure. Before: * nread <= 0, server closed connection, bailing After: * nread == 0, stream closed, bailing Closes https://github.com/curl/curl/pull/5118
* tool_setopt: correct the copyright year rangeDaniel Stenberg2020-03-181-1/+1
| | | | Follow-up to 5450428491
* schannel: add "best effort" revocation check optionJohannes Schindelin2020-03-1820-2/+136
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Implement new option CURLSSLOPT_REVOKE_BEST_EFFORT and --ssl-revoke-best-effort to allow a "best effort" revocation check. A best effort revocation check ignores errors that the revocation check was unable to take place. The reasoning is described in detail below and discussed further in the PR. --- When running e.g. with Fiddler, the schannel backend fails with an unhelpful error message: Unknown error (0x80092012) - The revocation function was unable to check revocation for the certificate. Sadly, many enterprise users who are stuck behind MITM proxies suffer the very same problem. This has been discussed in plenty of issues: https://github.com/curl/curl/issues/3727, https://github.com/curl/curl/issues/264, for example. In the latter, a Microsoft Edge developer even made the case that the common behavior is to ignore issues when a certificate has no recorded distribution point for revocation lists, or when the server is offline. This is also known as "best effort" strategy and addresses the Fiddler issue. Unfortunately, this strategy was not chosen as the default for schannel (and is therefore a backend-specific behavior: OpenSSL seems to happily ignore the offline servers and missing distribution points). To maintain backward-compatibility, we therefore add a new flag (`CURLSSLOPT_REVOKE_BEST_EFFORT`) and a new option (`--ssl-revoke-best-effort`) to select the new behavior. Due to the many related issues Git for Windows and GitHub Desktop, the plan is to make this behavior the default in these software packages. The test 2070 was added to verify this behavior, adapted from 310. Based-on-work-by: georgeok <giorgos.n.oikonomou@gmail.com> Co-authored-by: Markus Olsson <j.markus.olsson@gmail.com> Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de> Closes https://github.com/curl/curl/pull/4981
* multi: Improve parameter check for curl_multi_remove_handleJay Satiro2020-03-182-2/+6
| | | | | | | | | - If an easy handle is owned by a multi different from the one specified then return CURLM_BAD_EASY_HANDLE. Prior to this change I assume user error could cause corruption. Closes https://github.com/curl/curl/pull/5116
* windows: suppress UI in all CryptAcquireContext() callsViktor Szakats2020-03-176-13/+13
| | | | | | Ref: https://docs.microsoft.com/windows/win32/api/wincrypt/nf-wincrypt-cryptacquirecontexta#parameters Reviewed-by: Marc Hörsken Closes https://github.com/curl/curl/pull/5088
* writeout_json: add missing comma to fix the HTTP versionDaniel Stenberg2020-03-171-1/+1
| | | | Follow-up to 04c03416e68fd635a15
* test 970: verify --write-out '%{json}'Daniel Stenberg2020-03-173-3/+136
| | | | | | | | | | | Makes curl_easy_getinfo() of "variable" numerical content instead return the number set in the env variable `CURL_TIME`. Makes curl_version() of "variable" textual content. This guarantees a stable version string which can be tested against. Environment variable `CURL_VERSION` defines the content. Assisted-by: Mathias Gumz
* writeout: support to generate JSON outputMathias Gumz2020-03-178-83/+361
| | | | | | | | | | | | | | | | This commit adds support to generate JSON via the writeout feature: -w "%{json}" It leverages the existing infrastructure as much as possible. Thus, generating the JSON on STDERR is possible by: -w "%{stderr}%{json}" This implements a variant of https://github.com/curl/curl/wiki/JSON#--write-out-json. Closes #4870
* CI: stop ignoring 323, it is disabledDaniel Stenberg2020-03-173-15/+10
|
* DISABLED: disable test 323Daniel Stenberg2020-03-171-0/+3
| | | | | | | | | | | | | The test uses SRP to "a server not supporting it" but modern stunnel versions will silently accept it and remain happy. The test is therefore faulty. I haven't figured out how to make stunnel explicitly reject SRP-using connects. Reported-by: Marc Hörsken Fixes #5105 Closes #5113
* ci/tests: increase timeouts for torture builds on Azure PipelinesMarc Hoersken2020-03-171-2/+2
| | | | | | For some reason the torture builds have slowed down recently. Reported-by: Daniel Stenberg