Skip to content

Added set_curl_options method to allow custom options#407

Merged
mblaney merged 1 commit intosimplepie:masterfrom
Barnetik:master
Mar 12, 2016
Merged

Added set_curl_options method to allow custom options#407
mblaney merged 1 commit intosimplepie:masterfrom
Barnetik:master

Conversation

@arraintxo
Copy link
Contributor

We were having troubles with self-signed https hosts and have seen more people with the same issue, so we modified SimplePie to simply allow custom curl_options to be passed as configuration params.

Usage:

            $simplePie = new SimplePie();
            $simplePie->set_curl_options(
                array(
                    CURLOPT_SSL_VERIFYHOST => false,
                    CURLOPT_SSL_VERIFYPEER => false
                )
            );

@mblaney
Copy link
Member

mblaney commented May 26, 2015

thanks @arraintxo!

Alkarex added a commit to Alkarex/simplepie that referenced this pull request Jul 11, 2015
Added set_curl_options method to allow custom options

Conflicts:
	library/SimplePie.php
	library/SimplePie/File.php

Choose the generic approach of adding any parameter to cURL
simplepie#407
Barnetik@51f9f08

instead of the approach focusing only on proxy aspects
simplepie#360
simplepie#359

FreshRSS/FreshRSS@9aab83a
Alkarex added a commit to Alkarex/FreshRSS that referenced this pull request Jul 11, 2015
silent Warning cast with PHP 5.3.3 (RHEL-6)
simplepie/simplepie#258
Use rss:pubDate over atom:updated when determining the posting date
simplepie/simplepie#288
Update Item.php: add support for media thumbnails
simplepie/simplepie#340
Remove PHP Notices simplepie/simplepie#353
And removed "FreshRSS" annotations for pull requests submitted upstream

Fixed a typo in Date.php (wrong number for September)
simplepie/simplepie#374
Remove duplicate case simplepie/simplepie#406
Added set_curl_options method to allow custom options
simplepie/simplepie#407
Improved error message simplepie/simplepie#412
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Alkarex
Copy link
Contributor

Alkarex commented Jul 12, 2015

Hello @arraintxo,
There are a few other calls to the File constructor, which may also need to be updated with your new $this->curl_options parameter:

What do you think?

@samwilson
Copy link

+1 this would be a useful feature.

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