Skip to content

Conversation

@bagder
Copy link
Member

@bagder bagder commented Oct 18, 2025

In init_config_builder_verifier() the call to
rustls_root_cert_store_builder_build() set result on failure but did not return.

Pointed out by ZeroPath

In init_config_builder_verifier() the call to
rustls_root_cert_store_builder_build() set result on failure but did not
return.

Pointed out by ZeroPath
@github-actions github-actions bot added the TLS label Oct 18, 2025
@bagder bagder marked this pull request as ready for review October 18, 2025 21:37
@bagder
Copy link
Member Author

bagder commented Oct 18, 2025

/cc @cpu

Copy link
Contributor

@cpu cpu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! This fix looks good to me. Another point for the AI bot ☑️

@bagder bagder closed this in d3b2ba9 Oct 19, 2025
@bagder bagder deleted the bagder/rustls-early-exit branch October 19, 2025 21:48
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.

2 participants