Skip to content

wolfssl: document v5.0.0 (2021-11-01) as minimum required#21080

Closed
vszakats wants to merge 4 commits intocurl:masterfrom
vszakats:wolfssldate
Closed

wolfssl: document v5.0.0 (2021-11-01) as minimum required#21080
vszakats wants to merge 4 commits intocurl:masterfrom
vszakats:wolfssldate

Conversation

@vszakats
Copy link
Copy Markdown
Member

@vszakats vszakats commented Mar 23, 2026

The previously documented version was based on version checks made in
the source. In practice though, curl doesn't build with <5.0.0.

Also:

  • bump main internal version check.
  • drop superfluous internal version checks.

  • build also requires a wolfSSL built with --enable-opensslextra. Document this? Drop logic dealing with it missing? Fix to build without? → [FUTURE PR]
  • drop logic dealing with wolfSSL <5.0.0.

Bumped from 4.8.0 in 2024 with the introduction of CA caching in
80aa519, due to:

lib/vtls/wolfssl.c:776:6: error: call to undeclared function 'wolfSSL_X509_STORE_up_ref'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration]
  776 |   if(wolfSSL_X509_STORE_up_ref(store)) {
      |      ^

Tested with builds --enable-curl --enable-opensslextra.

@vszakats vszakats changed the title wolfssl: document 5.0.0+ (2021-11-01) as minimum required wolfssl: document 5.0.0 (2021-11-01) as minimum required Mar 24, 2026
@vszakats vszakats changed the title wolfssl: document 5.0.0 (2021-11-01) as minimum required wolfssl: document v5.0.0 (2021-11-01) as minimum required Mar 24, 2026
@vszakats vszakats marked this pull request as draft March 24, 2026 00:31
@vszakats vszakats closed this in 372d721 Mar 24, 2026
@vszakats vszakats deleted the wolfssldate branch March 24, 2026 10:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

1 participant