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

ripcord::xmlrpcClient, with https url #3

Open
GoogleCodeExporter opened this issue Jul 7, 2015 · 2 comments
Open

ripcord::xmlrpcClient, with https url #3

GoogleCodeExporter opened this issue Jul 7, 2015 · 2 comments

Comments

@GoogleCodeExporter
Copy link

hai..
thank you so much for build ripcord,
very very nice..


owh ya, i have a problem with https url..
in my project, i've change the method post on Ripcord_Transport_Stream 
implements Ripcord_Transport..


to: (see attachment)

this is just my suggestion :-)
once again.. thank you very much for ripcord.

Original issue reported on code.google.com by mr.ryansilalahi on 13 Aug 2012 at 5:18

Attachments:

@GoogleCodeExporter
Copy link
Author

Hi, there is a Ripcord_Transport_CURL class in ripcord as well. You can use it 
by calling Ripcord::client() like this:

$client = ripcord::client( $url, $options, new Ripcord_Transport_CURL( array(
    CURLOPT_VERBOSE => TRUE,
    CURLOPT_TIMEOUT => 60,
    CURLOPT_CONNECTTIMEOUT => 60,
    CURLOPT_MAXREDIRS => 3,
    CURLOPT_FOLLOWLOCATION => 1,
    CURLOPT_HTTPHEADER => array('Content-type: text/xml'),
    CURLOPT_SSL_VERIFYTYPEER => false
) ) );

I guess the HTTPHEADER should have been standard :)
The default class also returns the headers as a string instead of an array, 
have to look into that as well.

Anyway, I hope this works for you. If not tell me whats missing and I'll fix it.

~
Auke

Original comment by a...@muze.nl on 10 Dec 2012 at 11:39

@GoogleCodeExporter
Copy link
Author

TODO: 
- add the CURLOPT_HTTPHEADER with Content-type: text/xml as default in the CURL 
transport layer
- check the other CURLOPT settings for a sane default behaviour.

Original comment by a.c.van....@gmail.com on 10 Dec 2012 at 12:23

  • Changed state: Accepted

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant