Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add
httpx.SSLContext
configuration. #3022Add
httpx.SSLContext
configuration. #3022Changes from 38 commits
88f2ab7
58e9fdc
1afe2c9
6be802e
d124d8f
6f1162b
f613dee
55ec002
7346bc2
7f35134
8d5983a
f759dec
7227573
0567aab
2d79640
4d84953
df7bf31
93ca75f
2b7cf9e
fa065dd
f09a823
4eba150
f3b1dc7
998f445
a08ff77
827ae5b
b3673bd
d3a8aab
808a891
289cf9c
6a85f2c
d235e76
1a79bd9
3f9aaa8
46268b7
26de9d0
49545bb
01bb388
13cb957
51ec619
e7c607c
f894a06
cb71af3
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
also needs remove
cert
andverify
here:httpx/docs/compatibility.md
Line 202 in 1bf1fc0
Do we need also mention Httpx doesn't support
cert
andverify
while Requests does?There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed, thanks
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
perhaps it's better link to this section instead of removing?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We removed those support, do we need to mention them?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, I think. Since it's removed now, it will be a thing that Requests has but we don't.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we still need this part
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It was moved to the ssl documentation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
does it make sense to have it in environment_variables.md with linking its description to ssl documentation?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it would be better to get rid of environment_variables.md. It's better to show env variables in each section, and only related ones.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think that'll work out best yep. Let's work on that as a follow-up.