-
Notifications
You must be signed in to change notification settings - Fork 95
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
Allow to use HTTPS #43
Comments
Yes, that approach should work. A bit strange to reuse the HTTP transport, and it requires you to add another constructor param to it. Its probably better to just extend it and change only the protocol part. |
Yes, but HTTP is already extend with Base. What was your idea? |
will we see HTTPs support as a feature soon? |
Yes, I have a thought @kcampion $client = Client::connection(array(
...
'secure' => true
)); Then each transport can infer what |
I updated source code to allow using HTTPS. I'm on the right way?
kcampion@f0be154
The text was updated successfully, but these errors were encountered: