-
Notifications
You must be signed in to change notification settings - Fork 271
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
Can't test against snakeoil website #355
Comments
Agreed. This is an issue with quite a number of vendor SDKs that I use, especially since php-5.6 |
The way I would solve this in general is to use a HTTP request library, but here we are trying to keep dependencies to a minimum so the proposed solution. |
Would you prefer a implicit stream context (http://php.net/manual/en/function.stream-context-set-default.php) or an explicit one like this file_get_contents($url, false, $streamContext) ? |
The first one implies global state so the second is better imho. |
See #360. |
It would be nice if the user could set a context for the file_get_contents() call in RemoteCoverage.php. This way they could create a context that sets verify_peer and verify_peer_name to false. This error is silently ignored due to @.
The text was updated successfully, but these errors were encountered: