Skip to content

Load SSL Context on init#709

Merged
lovelydinosaur merged 1 commit into
masterfrom
load-ssl-context-on-init
Jan 2, 2020
Merged

Load SSL Context on init#709
lovelydinosaur merged 1 commit into
masterfrom
load-ssl-context-on-init

Conversation

@lovelydinosaur
Copy link
Copy Markdown
Contributor

Follow up to #706

Load the SSLContext at the point that SSLConfig is initialized, rather than per-request.

One benefit of this is that we'll now raise any configuration errors that the point of client-init, rather than on the initial request.

@lovelydinosaur lovelydinosaur added user-experience Ensuring that users have a good experience using the library refactor Issues and PRs related to code refactoring labels Jan 2, 2020
Copy link
Copy Markdown
Contributor

@florimondmanca florimondmanca left a comment

Choose a reason for hiding this comment

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

Nice!

@lovelydinosaur lovelydinosaur merged commit 79a9748 into master Jan 2, 2020
@lovelydinosaur lovelydinosaur deleted the load-ssl-context-on-init branch January 2, 2020 16:52
Kludex referenced this pull request in pydantic/httpx2 May 11, 2026
Bumps [mkdocstrings[python-legacy]](https://github.com/mkdocstrings/mkdocstrings) from 0.19.1 to 0.22.0.
- [Release notes](https://github.com/mkdocstrings/mkdocstrings/releases)
- [Changelog](https://github.com/mkdocstrings/mkdocstrings/blob/master/CHANGELOG.md)
- [Commits](mkdocstrings/mkdocstrings@0.19.1...0.22.0)

---
updated-dependencies:
- dependency-name: mkdocstrings[python-legacy]
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

refactor Issues and PRs related to code refactoring user-experience Ensuring that users have a good experience using the library

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants