Skip to content

Add missing WOLFSSL_HOSTNAME_VERIFY_ALT_NAME_ONLY guards#9373

Merged
douzzer merged 2 commits intowolfSSL:masterfrom
julek-wolfssl:WOLFSSL_HOSTNAME_VERIFY_ALT_NAME_ONLY
Nov 8, 2025
Merged

Add missing WOLFSSL_HOSTNAME_VERIFY_ALT_NAME_ONLY guards#9373
douzzer merged 2 commits intowolfSSL:masterfrom
julek-wolfssl:WOLFSSL_HOSTNAME_VERIFY_ALT_NAME_ONLY

Conversation

@julek-wolfssl
Copy link
Copy Markdown
Member

No description provided.

@kcghost
Copy link
Copy Markdown

kcghost commented Nov 5, 2025

Wouldn't this print a misleading error message? The remaining error message indicates a check against CN took place, but failed. Rather than checks against CN being disabled and the SAN check failing.

SparkiDev
SparkiDev previously approved these changes Nov 5, 2025
@julek-wolfssl
Copy link
Copy Markdown
Member Author

Thanks @kcghost, I updated the logs to match what is happening.

@douzzer douzzer merged commit 9e9a739 into wolfSSL:master Nov 8, 2025
339 of 340 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: Possible issue with WOLFSSL_HOSTNAME_VERIFY_ALT_NAME_ONLY

5 participants