#
1a2d38c4 |
| 29-Oct-2024 |
Viktor Szakats |
GHA/windows: avoid curl.exe libtool wrapper Avoid the `curl.exe` wrapper binary created by libtool, and run the real `curl.exe` directly for tests and version information. This
GHA/windows: avoid curl.exe libtool wrapper Avoid the `curl.exe` wrapper binary created by libtool, and run the real `curl.exe` directly for tests and version information. This solution was used in Azure jobs. I missed it when migrating jobs to GHA. Applies to tests run in the `mingw, AM x86_64 c-ares U` job, which has seen unexplained flakiness. Ref: 354afc891df4b60b8017fc5d35a05daedb2cd812 #6049 Follow-up to e53523fef07894991c69d907a7c7794c7ada4ff4 #14859 Closes #15437
show more ...
|
#
ac7ae08f |
| 08-Sep-2024 |
Jay Satiro |
schannel: reclassify extra-verbose schannel_recv messages - Create a new macro SCH_DEV() to manage verbose debug messages that are only useful for debugging Schannel recv decryption.
schannel: reclassify extra-verbose schannel_recv messages - Create a new macro SCH_DEV() to manage verbose debug messages that are only useful for debugging Schannel recv decryption. schannel_recv contains a lot of useful debug messages to help debug the function, however in practice they are not otherwise useful and showing them in debug builds adds a lot of noise. To show these messages curl must now be built with CURL_SCHANNEL_DEV_DEBUG defined. Prior to this change many, but not all, extra-verbose messages were wrapped in DEBUGF() so they were only shown in debug builds. Ref: https://github.com/curl/curl/issues/14807 Closes #14826
show more ...
|
#
38c57bdf |
| 19-Oct-2024 |
renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> |
GHA: update five dependencies - rojopolis/spellcheck-github-actions digest to ab8ac45 - nghttp2/nghttp2 to v1.64.0 - actions/cache digest to 6849a64 - github/codeql-action digest
GHA: update five dependencies - rojopolis/spellcheck-github-actions digest to ab8ac45 - nghttp2/nghttp2 to v1.64.0 - actions/cache digest to 6849a64 - github/codeql-action digest to 6624720 - Update actions/checkout digest to 11bd719 Closes #15341 Closes #15346 Closes #15365 Closes #15366 Closes #15387
show more ...
|
#
5f9411f9 |
| 23-Oct-2024 |
Viktor Szakats |
GHA/windows: work around Git for Windows perf regression Fix the significant perf regression for vcpkg jobs by switching to the MSYS2 shell environment from Git for Windows. This env is
GHA/windows: work around Git for Windows perf regression Fix the significant perf regression for vcpkg jobs by switching to the MSYS2 shell environment from Git for Windows. This env is already used for old-mingw-w64 job that remained unaffected by this issue. The issue began with the windows-runner update 20241015.1.0. It bumped Git for Windows from Git 2.46.2.windows.1 to Git 2.47.0.windows.1. GfW bumped its MSYS2 components, including `msys-2.0.dll`. That's Cygwin code, which may have contributed to this. Pipes were involved and `runtests.pl` relies on pipes heavily in parallel mode. (The issue was not seen with parallel tests disabled, in retrospect.) This is useful as a permanent solution too. It drop GfW as a dependency and makes Windows jobs use one less shell/env flavour. Long term it might help to use native Windows Perl to avoid the MSYS layer completely, if there is a way to make that work. Assortment of possibly related links: https://cygwin.com/pipermail/cygwin/2024-August/256398.html https://github.com/cygwin/cygwin/commit/f78009cb1ccf84cc343cf2441c76196461d87532 https://github.com/cygwin/cygwin/commit/7f3c22532577ae0a926e8eb8ad63787c9841abbf https://github.com/actions/runner-images/issues/10843 https://github.com/git-for-windows/git/issues/5199 https://github.com/git-for-windows/msys2-runtime/pull/75 https://github.com/git-for-windows/msys2-runtime/commit/7913a41703dbc476ad3cf1b85e6939ebbe524251 https://github.com/git-for-windows/msys2-runtime/commit/555afcb2f3a6638084912ce1011bd6acef59ea79 https://github.com/cygwin/cygwin/commit/1c5f4dcdc5ec3344e3fd741c43fa359d0e1323c0 Follow-up to c33174d42fc8a4a0625b46f1d09f5e79eb2abbf1 #15364 Follow-up to 1e0305973c22b1d84036fe0c4eee34aea5cd40cc #15356 Closes #15380
show more ...
|
#
c33174d4 |
| 22-Oct-2024 |
Viktor Szakats |
GHA/windows: increase timeout for vcpkg jobs due to slowness The openssl job no longer fits into 10 minutes since the 20241015.1.0 GHA windows-latest image update. This caused all runs t
GHA/windows: increase timeout for vcpkg jobs due to slowness The openssl job no longer fits into 10 minutes since the 20241015.1.0 GHA windows-latest image update. This caused all runs to fail. The `run tests` step takes ~10 minutes now, up from ~4. This is 6 minutes more than before these updates. It's seen with other vcpkg jobs too, tests run slower than half speed since. Bump the timeout to make it, though the headroom is now less than it was. Before: https://github.com/curl/curl/actions/runs/11386748199/job/31679733295 https://github.com/curl/curl/actions/runs/11347976608/job/31560690219 After: https://github.com/curl/curl/actions/runs/11462332743/job/31893491625?pr=15364 Ref: https://github.com/actions/runner-images/commit/fcc4cdb1d095af1317859c4809364538953b3497 Ref: https://github.com/curl/curl/pull/15335#issuecomment-2423759953 Follow-up to 1e0305973c22b1d84036fe0c4eee34aea5cd40cc #15356 Closes #15364
show more ...
|
#
c2e26367 |
| 19-Oct-2024 |
Tal Regev |
GHA/windows: add http3 to libressl vcpkg job Also drop a libressl workaround no longer necessary. Closes #15338
|
#
1e030597 |
| 22-Oct-2024 |
Viktor Szakats |
GHA/windows: ignore results for test 987 987 is `SMTPS with redundant explicit SSL request`. Root cause undiscovered. Started failing after GHA bumping the windows image to
GHA/windows: ignore results for test 987 987 is `SMTPS with redundant explicit SSL request`. Root cause undiscovered. Started failing after GHA bumping the windows image to `20241015.1.0`: https://github.com/actions/runner-images/commit/fcc4cdb1d095af1317859c4809364538953b3497 https://github.com/actions/runner-images/pull/10803 vcpkg packages also got bumped as a result. They seem unrelated: c-ares 1.33.1 -> 1.34.1 nghttp2 1.62.1 -> 1.63.0 nghttp3 1.5.0 -> 1.6.0 (there may be more) Ref: https://github.com/curl/curl/pull/15335#issuecomment-2423759953 Closes #15356
show more ...
|
#
e8a007de |
| 12-Oct-2024 |
Viktor Szakats |
GHA: optimize test prereq steps - Linux: move test and pytest prereqs right before test run. - returns build phase results faster. - allows skipping steps for jobs that don't nee
GHA: optimize test prereq steps - Linux: move test and pytest prereqs right before test run. - returns build phase results faster. - allows skipping steps for jobs that don't need them. - makes dependencies more transparent. - sync prereq install step names. - use `tests/requirements.txt` more. Closes #15275
show more ...
|
#
66cc0157 |
| 05-Oct-2024 |
Tal Regev |
GHA/windows: drop vcpkg workaround Reverts 841f42150ae7b9f903cc5d266cec60a8090c0f27 #15133 Closes #15217
|
#
ba68eb02 |
| 08-Oct-2024 |
renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> |
CI: update rojopolis/spellcheck, actions/checkout, actions/upload-artifact - update rojopolis/spellcheck-github-actions digest to 7ff888c - update actions/checkout digest to eef6144
CI: update rojopolis/spellcheck, actions/checkout, actions/upload-artifact - update rojopolis/spellcheck-github-actions digest to 7ff888c - update actions/checkout digest to eef6144 - update actions/upload-artifact digest to b4b15b8 Closes #15207 Closes #15184 Closes #15187
show more ...
|
#
7d53a592 |
| 07-Oct-2024 |
dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> |
CI: bump github/codeql-action, vmactions/omnios-vm and actions/cache - bump github/codeql-action from 3.26.10 to 3.26.11 - bump vmactions/omnios-vm from 1.0.7 to 1.0.8 - bump actions
CI: bump github/codeql-action, vmactions/omnios-vm and actions/cache - bump github/codeql-action from 3.26.10 to 3.26.11 - bump vmactions/omnios-vm from 1.0.7 to 1.0.8 - bump actions/cache from 4.0.2 to 4.1.1 Closes #15178 Closes #15179 Closes #15244
show more ...
|
#
7bff6864 |
| 11-Oct-2024 |
Viktor Szakats |
ci: dump `curl_config.h` to log in all jobs Also: - GHA/windows: merge full and brief dump into a single job step. - fix shellcheck warning 'useless cat'. Closes #15266
|
#
ae5e538e |
| 10-Oct-2024 |
Viktor Szakats |
GHA: drop `--parallel` option for CMake + Ninja jobs It's managed by Ninja automatically. Closes #15249
|
#
b941d16d |
| 08-Oct-2024 |
Viktor Szakats |
GHA/windows: drop vcpkg shiftmedia-gnutls, replace with mbedtls GnuTLS vcpkg package broken again with the latest runner image update: https://github.com/curl/curl/actions/runs/112400113
GHA/windows: drop vcpkg shiftmedia-gnutls, replace with mbedtls GnuTLS vcpkg package broken again with the latest runner image update: https://github.com/curl/curl/actions/runs/11240011311/job/31248406051?pr=15203#step:5:137 Previous breakage: 09b21e4755b4cc8ff31e9126aa9caf464988551e #14864 Closes #15204
show more ...
|
#
841f4215 |
| 03-Oct-2024 |
Tal Regev |
GHA/windows: add workaround for upstream vcpkg issue Fixes: ``` error: https://github.com/google/brotli/archive/v1.1.0.tar.gz: WinHttpSendRequest failed with exit code 10106 ```
GHA/windows: add workaround for upstream vcpkg issue Fixes: ``` error: https://github.com/google/brotli/archive/v1.1.0.tar.gz: WinHttpSendRequest failed with exit code 10106 ``` Apply workaround: https://github.com/microsoft/vcpkg/issues/41199#issuecomment-2378255699 Bug commit: https://github.com/microsoft/vcpkg-tool/commit/d3fc35774f86782f356ef0f9352ba857fa57302e#diff-33cf7c311a76d4a838f91b078c2f8cbc984557379f7b345a268ec6deb665a91eR666-R675 Fix commit: https://github.com/microsoft/vcpkg-tool/commit/3122da72b1e4bf98bfc3d51937e6ad6cc4292cdf Fix PR: https://github.com/microsoft/vcpkg-tool/pull/1501 Fixes #15128 Reported-by: Viktor Szakats Closes #15133
show more ...
|
#
303c0cf7 |
| 30-Sep-2024 |
dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> |
CI: bump actions/checkout from 4.1.7 to 4.2.0 Bumps [actions/checkout](https://github.com/actions/checkout) from 4.1.7 to 4.2.0. - [Release notes](https://github.com/actions/checkout/rel
CI: bump actions/checkout from 4.1.7 to 4.2.0 Bumps [actions/checkout](https://github.com/actions/checkout) from 4.1.7 to 4.2.0. - [Release notes](https://github.com/actions/checkout/releases) - [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md) - [Commits](https://github.com/actions/checkout/compare/692973e3d937129bcbf40652eb9f2f61becf3332...d632683dd7b4114ad314bca15554477dd762a938) --- updated-dependencies: - dependency-name: actions/checkout dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Closes #15095
show more ...
|
#
aed3f023 |
| 30-Sep-2024 |
Viktor Szakats |
GHA: move Cygwin jobs back into the Windows workflow Cygwin jobs took 30 minutes and was the reason for moving them to their own workflow. After recent changes, they finish in 8 minutes,
GHA: move Cygwin jobs back into the Windows workflow Cygwin jobs took 30 minutes and was the reason for moving them to their own workflow. After recent changes, they finish in 8 minutes, which is on par or lower than the longest Windows jobs. Thus, Cygwin can now be moved back to the Windows workflow without causing extra wait for all the workflow jobs to complete (and allowing a manual retry for the remaining flaky jobs). Notice that Cygwin still uses the cygwin/cygwin-install-action action to install Cygwin itself. This action uses no caching and the time it takes fluctuates widely depending on mirror speed, congestion and other things. If Cygwin tends to hold up jobs again for this or other reasons, this patch may be reverted. Follow-up to 80a8e24956ff652e865113231649cb0c14f1e7c2 #15083 Follow-up to 2d8464c4cb9736144946701c5e632c817888eaec #14366 Closes #15086
show more ...
|
#
cd636297 |
| 27-Sep-2024 |
Daniel Stenberg |
CI/winbuild: remove enabling of websocket - done by default now Closes #15076
|
#
80a8e249 |
| 29-Sep-2024 |
Viktor Szakats |
GHA/cygwin, msys: move tests to cmake jobs, to finish faster Move test runs from autotools jobs to cmake ones for Cygwin and MSYS. This makes producing test results and finishing the wor
GHA/cygwin, msys: move tests to cmake jobs, to finish faster Move test runs from autotools jobs to cmake ones for Cygwin and MSYS. This makes producing test results and finishing the workflows faster, because the cmake build steps are much faster than autotools in these envs. Also: - drop building examples with Cygwin autotools. It takes almost 4 minutes. Keep building them with cmake, taking 20 seconds. - drop building examples in MSYS autotools jobs that run tests. Keep building them in autotools jobs without tests. It makes the longest running job 2 minutes shorter. After this patch Cygwin job times are on par with or lower than Windows ones. It means Cygwin doesn't cause extra delay to finish the whole workflow, allowing to re-merge these jobs into the Windows workflow. Closes #15083
show more ...
|
#
47d604ae |
| 29-Sep-2024 |
Viktor Szakats |
GHA/windows: fix `find` in old-mingw-w64 `curl -V` step By default it runs the Windows `find` command and fails. Tweak the `PATH` to find the MSYS shell implementation.
|
#
2c419fc1 |
| 28-Sep-2024 |
Viktor Szakats |
ci: tidy-ups - http3-linux: add newlines for readability. - http3-linux: use `make pytest` to run pytest. - checksrc: use `|` multiline to sync with other uses. - checksrc: prefe
ci: tidy-ups - http3-linux: add newlines for readability. - http3-linux: use `make pytest` to run pytest. - checksrc: use `|` multiline to sync with other uses. - checksrc: prefer `$()`. - prefer `>-` over `>` for folded option lists. - cygwin: drop a `grep` no longer necessary. Closes #15081
show more ...
|
#
9541e666 |
| 23-Sep-2024 |
Viktor Szakats |
GHA/windows: formatting, adjust timeouts, tidy-ups - reduce job timeouts for msys2 and old-mingw-w64 jobs. - fold long lines and reorder options in MSVC jobs. - make some options def
GHA/windows: formatting, adjust timeouts, tidy-ups - reduce job timeouts for msys2 and old-mingw-w64 jobs. - fold long lines and reorder options in MSVC jobs. - make some options default in MSVC jobs. Closes #15017
show more ...
|
#
7f3d5982 |
| 25-Sep-2024 |
Stefan Eissing |
CI: improvements in test reliability and performance - CI default: raise parallelism to 20 - CI valgind: set parallelism to 6 - CI non-native: adapt parallelism for OS builds - C
CI: improvements in test reliability and performance - CI default: raise parallelism to 20 - CI valgind: set parallelism to 6 - CI non-native: adapt parallelism for OS builds - CI Windows: no longer ignore FTP, TFTP, MQTT and SMTP - CI Windows: restrict test timeout to 10 minutes - CI Windows: do not run tests for msh3 build - tests, various: restrict curl invocation to ipv4 to avoid talking to any ipv6 test server running in parallel - tests: requiring http/2 server no longer needs to mention http server - test 190: use a fixed timeout of 10 seconds instead of %FTPTIME2, as that value rises under parallel load beyond what the server is waiting and then produces different results - test 1540: add debug logging - testrunner: add verify checks for http/2 and http/3 server Closes #15040
show more ...
|
#
d48747b2 |
| 22-Sep-2024 |
Viktor Szakats |
GHA/windows: mark 3023, 3024 flaky for mingw-w64 7.3.0 job They became flaky recently as reported by Testclutch and firsthand observations. It's curious because it only seems to happen i
GHA/windows: mark 3023, 3024 flaky for mingw-w64 7.3.0 job They became flaky recently as reported by Testclutch and firsthand observations. It's curious because it only seems to happen in 7.3.0, but not in the very similar (except non-Unicode) 9.5.0 job. We've found no explanation or reason for them so far. ``` test 3023...[HTTPS GET to localhost, first subject alt name matches, CN does not match (Schannel variant)] 3023: protocol FAILED! There was no content at all in the file log/14/server.input. Server glitch? Total curl failure? Returned: 7 ``` Ref: https://github.com/curl/curl/actions/runs/10871191391/job/30164710777?pr=14918#step:13:4849 ``` test 3024...[HTTPS GET to localhost, last subject alt name matches, CN does not match (Schannel variant)] 3024: protocol FAILED! There was no content at all in the file log/8/server.input. Server glitch? Total curl failure? Returned: 7 ``` Ref: https://github.com/curl/curl/actions/runs/10871191391/job/30164710777?pr=14918#step:13:4713 ``` FAIL 3023: 'HTTPS GET to localhost, first subject alt name matches, CN does not match (Schannel variant)' HTTPS, HTTP GET, PEM certificate FAIL 3024: 'HTTPS GET to localhost, last subject alt name matches, CN does not match (Schannel variant)' HTTPS, HTTP GET, PEM certificate ``` https://github.com/curl/curl/actions/runs/10982991876/job/30491895264?pr=14930#step:14:4914 https://github.com/curl/curl/actions/runs/10871191391/job/30164710777?pr=14918 https://github.com/curl/curl/actions/runs/10841065505/job/30084573629?pr=14859#step:13:5484 https://github.com/curl/curl/actions/runs/10858001821/job/30135376138?pr=14906#step:13:4841 Reported-by: Testclutch Fixes https://github.com/curl/curl/pull/14905#issuecomment-2350772804 Ref: https://github.com/curl/curl/discussions/14854#discussioncomment-10652044 Closes #15006
show more ...
|
#
22652a5a |
| 18-Aug-2024 |
Viktor Szakats |
curl: add options for safe/no CA bundle search (Windows) Add `CURL_CA_SEARCH_SAFE` build-time option to enable CA bundle search in the `curl` tool directory. The lookup method was alread
curl: add options for safe/no CA bundle search (Windows) Add `CURL_CA_SEARCH_SAFE` build-time option to enable CA bundle search in the `curl` tool directory. The lookup method was already used to find `.curlrc` and `_curlrc` (on Windows). On Windows it overrides the unsafe default `SearchPath()` method. Enable with: - cmake: `-DCURL_CA_SEARCH_SAFE=ON` - autotools: `--enable-ca-search-safe` - raw: `CPPFLAGS=-DCURL_CA_SEARCH_SAFE` On Windows, before this patch the whole `PATH` was searched for a CA bundle. `PATH` may contain unwanted or world-writable locations, including the current directory. Searching them all is convenient to pick up any CA bundle, but not secure. The Muldersoft curl distro implements such CA search via a custom patch for Windows: https://github.com/lordmulder/cURL-build-win32/blob/cd652d4792c177c98b08b4309d3cac2b8dbbf9b0/patch/curl_tool_doswin.diff#L50 MSYS2/mingw-w64 distro has also been rolling a patch solving this: https://github.com/msys2/MINGW-packages/blob/master/mingw-w64-curl/0001-Make-cURL-relocatable.patch https://github.com/msys2/MINGW-packages/blob/master/mingw-w64-curl/pathtools.c Also add option to fully disable Windows CA search: - cmake: `-DCURL_DISABLE_CA_SEARCH=ON` - autotools: `--disable-ca-search` - raw: `CPPFLAGS=-DCURL_DISABLE_CA_SEARCH`. Both options are considered EXPERIMENTAL, with possible incompatible changes or even (partial) removal in the future, depending on feedback. An alternative, secure option is to embed the CA bundle into the binary. Safe search can be extended to other platforms if necessary or useful, by using `_NSGetExecutablePath()` (macOS), `/proc/self/exe` (Linux/Cygwin), or `argv[0]`. Closes #14582
show more ...
|