Skip to content
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 httpbin_ca_bundle #8

Merged
merged 2 commits into from
Dec 2, 2014
Merged

Add httpbin_ca_bundle #8

merged 2 commits into from
Dec 2, 2014

Conversation

t-8ch
Copy link
Contributor

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

The name is a bit unwieldy...

@msabramo
Copy link
Collaborator

msabramo commented Dec 2, 2014

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

@t-8ch
Copy link
Contributor Author

t-8ch commented Dec 2, 2014

@msabramo done

@msabramo
Copy link
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
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
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
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
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
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
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