Skip to content

Commit

Permalink
bug #58343 [HttpClient] Add crypto_method to scoped client options …
Browse files Browse the repository at this point in the history
…(HypeMC)

This PR was merged into the 6.4 branch.

Discussion
----------

[HttpClient] Add `crypto_method` to scoped client options

| Q             | A
| ------------- | ---
| Branch?       | 6.4
| Bug fix?      | yes
| New feature?  | no
| Deprecations? | no
| Issues        | -
| License       | MIT

While working on something else I've noticed that the `crypto_method` option wasn't added for scoped clients in #50274. I'm not sure if this was intentional or not, but as far as I can tell, there's no reason for the option to not be there, so I'm guessing it was an oversight.

Commits
-------

e274ee46cb7 [HttpClient] Add `crypto_method` to scoped client options
  • Loading branch information
nicolas-grekas committed Sep 28, 2024
2 parents bc9f770 + b7ed34d commit 72a2a25
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions DependencyInjection/Configuration.php
Original file line number Diff line number Diff line change
Expand Up @@ -2093,6 +2093,9 @@ private function addHttpClientSection(ArrayNodeDefinition $rootNode, callable $e
->variableNode('md5')->end()
->end()
->end()
->scalarNode('crypto_method')
->info('The minimum version of TLS to accept; must be one of STREAM_CRYPTO_METHOD_TLSv*_CLIENT constants.')
->end()
->arrayNode('extra')
->info('Extra options for specific HTTP client')
->normalizeKeys(false)
Expand Down

0 comments on commit 72a2a25

Please sign in to comment.