Skip to content

Use custom certificate if provided#5514

Merged
schlessera merged 6 commits into
wp-cli:masterfrom
WhiteWinterWolf:5111-tls-ca-validation
May 5, 2021
Merged

Use custom certificate if provided#5514
schlessera merged 6 commits into
wp-cli:masterfrom
WhiteWinterWolf:5111-tls-ca-validation

Conversation

@WhiteWinterWolf
Copy link
Copy Markdown
Contributor

@WhiteWinterWolf WhiteWinterWolf commented Apr 18, 2021

  • PHP setting 'curl.cainfo' enforces the CA file to use,
  • Otherwise fallback to system-wide defaults, then use the embedded CA file.

Fixes #5111

- PHP setting 'curl.cainfo' enforces the CA file to use,
- Otherwise fallback to system-wide defaults, then use the embedded CA file.
@WhiteWinterWolf WhiteWinterWolf requested a review from a team as a code owner April 18, 2021 09:44
@schlessera schlessera changed the title Enable the use of custom CA, fixes #5111 Use custom certificate if provided May 5, 2021
Comment thread php/utils.php Outdated
@schlessera schlessera added this to the 2.5.0 milestone May 5, 2021
@schlessera schlessera enabled auto-merge May 5, 2021 07:09
@schlessera
Copy link
Copy Markdown
Member

Thanks for the PR, @WhiteWinterWolf !

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

doesn't honour curl.cainfo setting

2 participants