Skip to content

Add httpbin_ca_bundle#8

Merged
msabramo merged 2 commits intokevin1024:masterfrom
t-8ch:master
Dec 2, 2014
Merged

Add httpbin_ca_bundle#8
msabramo merged 2 commits intokevin1024:masterfrom
t-8ch:master

Conversation

@t-8ch
Copy link
Copy Markdown
Contributor

@t-8ch t-8ch commented Aug 18, 2014

The name is a bit unwieldy...

@msabramo
Copy link
Copy Markdown
Collaborator

msabramo commented Dec 2, 2014

@t-8ch: Can you rebase to fix the merge conflicts?

@t-8ch
Copy link
Copy Markdown
Contributor Author

t-8ch commented Dec 2, 2014

@msabramo done

@msabramo
Copy link
Copy Markdown
Collaborator

msabramo commented Dec 2, 2014

@t-8ch: Awesome. Thank you! Could give me a short description of the benefit this provides? I was planning to try to do a release of pytest-httpbin 0.0.4 to PyPI and if this PR is ready, I might as well merge it, but I need to know what to put in the release notes.

@t-8ch
Copy link
Copy Markdown
Contributor Author

t-8ch commented Dec 2, 2014

@msabramo With this fixture there is no need to specify the bundle on every request.
And you don't have to care about where it is located.
Like 043b8bc.

@msabramo
Copy link
Copy Markdown
Collaborator

msabramo commented Dec 2, 2014

@t-8ch: Ah, I see, so it's a convenience for the test suite and therefore it probably doesn't have to be mentioned in the release notes, I think.

Thanks!

msabramo added a commit that referenced this pull request Dec 2, 2014
Add httpbin_ca_bundle
@msabramo msabramo merged commit 9d14ec7 into kevin1024:master Dec 2, 2014
@t-8ch
Copy link
Copy Markdown
Contributor Author

t-8ch commented Dec 2, 2014

It is also a convenience for the users, as they are using pytest-httpbin in the same way

@msabramo
Copy link
Copy Markdown
Collaborator

msabramo commented Dec 2, 2014

Oh right, good point!

msabramo added a commit to msabramo/pytest-httpbin that referenced this pull request Dec 2, 2014
@kevin1024
Copy link
Copy Markdown
Owner

Cool, probably want to mention in the read me they it will automatically set REQUESTS_CA_BUNDLE if using requests.

msabramo added a commit that referenced this pull request Dec 2, 2014
README.md: Add PR #8 to httpbin_ca_bundle change
msabramo added a commit to msabramo/pytest-httpbin that referenced this pull request Dec 3, 2014
for 0.0.4. Mention it will automatically set `REQUESTS_CA_BUNDLE` if
using requests.
@msabramo
Copy link
Copy Markdown
Collaborator

msabramo commented Dec 3, 2014

@kevin1024: Good idea => #21

msabramo added a commit that referenced this pull request Dec 3, 2014
README.md: Add a bit more info about PR #8
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants