Skip to content

runtests: pass config filename to stunnel in native format (Windows)#20413

Closed
vszakats wants to merge 1 commit intocurl:masterfrom
vszakats:cygwinstunnel
Closed

runtests: pass config filename to stunnel in native format (Windows)#20413
vszakats wants to merge 1 commit intocurl:masterfrom
vszakats:cygwinstunnel

Conversation

@vszakats
Copy link
Member

@vszakats vszakats commented Jan 23, 2026

Fixing (Seen when enabling stunnel for Cygwin in CI):

=== Start of file https_stunnel.log
 [ ] Initializing inetd mode configuration
 [ ] Running on Windows 6.2
[...]
 [.] Reading configuration from file /cygdrive/d/a/curl/curl/bld/tests/log/6/server/https_stunnel.conf
 [!] Cannot open configuration file

Ref: https://github.com/curl/curl/actions/runs/21289677523/job/61279662459?pr=20410

Cherry-picked from #20410

Seen when enabling stunnel for Cygwin in CI.

Fixing:
```
=== Start of file https_stunnel.log
 [ ] Initializing inetd mode configuration
 [ ] Running on Windows 6.2
[...]
 [.] Reading configuration from file /cygdrive/d/a/curl/curl/bld/tests/log/6/server/https_stunnel.conf
 [!] Cannot open configuration file
```
Ref: https://github.com/curl/curl/actions/runs/21289677523/job/61279662459?pr=20410

Cherry-picked from curl#20410
@vszakats vszakats added TLS tests Windows Windows-specific labels Jan 23, 2026
@vszakats vszakats closed this in d176f58 Jan 23, 2026
@vszakats vszakats deleted the cygwinstunnel branch January 23, 2026 15:52
vszakats added a commit that referenced this pull request Jan 23, 2026
Replace Chocolatey install with direct download and unpack. To reduce
CI dependencies (Chocolatey, NuGet), improve install performance
(10s -> 1s) and hopefully reliability. Last but not least to enable it
for the Cygwin CI job.

Caveats:
- Need to bump stunnel versions manually (2-3 times a year).
  Renovate could likely do it, but I failed to understand its
  documentation and miss tooling/interface to make tests.
- FIPS not enabled. (can be done if necessary)
- Possibly losing checksum verification (not sure if Chocolatey did it
  automatically for this package.)

Also:
- Increase minimum tests by 100 for the Cygwin job.

Ref: #16819 (earlier attempt)
Ref: https://www.stunnel.org/archive/
Ref: https://www.githubstatus.com/incidents/cqb5hcy0gx18
Follow-up to d176f58 #20413
Follow-up to 19b1e44 #20409

Closes #20410
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Development

Successfully merging this pull request may close these issues.

1 participant