Updating API post to use CURL_CA_BUNDLE #1
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
If user has CURL_CA_BUNDLE environment variable defined, have API curl call use that path.
jlipps
added a commit
that referenced
this pull request
Dec 11, 2012
Updating API post to use CURL_CA_BUNDLE
|
Thanks for the contrib. |
|
On OSX I have an environment variable defined to point my command line curl to a specific CA bundle (up to date). Sauce labs' API URL requires an up to date bundle to verify the SSL cert. However, php by default doesn't use the same environment variable that the CLI curl does, adding this if statement fixes Sausage for me on OSX Mountain Lion. |
|
That was fast, thanks! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
If user has CURL_CA_BUNDLE environment variable defined, have API curl call use that path.