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

Implement HTTPS support as well as ability to pass arbitrary curl options #4

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jasonhildebrand
Copy link

I have added support for HTTPS streams as well as passing additional options to curl.

It is sometimes necessary to pass additional curl options, for example to connect to a service which uses a self-signed certificate. This lets users pass these options without modifying the code.

@insidew
Copy link

insidew commented May 11, 2018

still getting

SOAP-ERROR: Parsing WSDL: Couldn't load from 'https://xxxxx.com/?wsdl' : failed to load external entity "https://xxxxx.com/?wsdl"

@josefbehr
Copy link

I just tried this in a real project with NTLM auth over https, and it worked like a charm. Thanks for the Code @jasonhildebrand
Can this be merged upstream, @matejsvajger ?

@jasonhildebrand
Copy link
Author

Thanks @josefbehr for the comment. I'm still actively using this project too (with my patch). It would be nice to see it merged in.

@baumannsven
Copy link

@matejsvajger Can you make a statement? I need the ssl connection, too. I tested the pull request successfully.

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.

4 participants