Allow for passing curl options

[RomainGoncalves]

This PR aims to allow the user to pass along to the read method some curl options.

Scenario
If you want to read a feed from a platform/website that requires authentication, you can't at the moment.
By passing curl options manually to the read method, the user can add any tools that they would require per call.

[vedmant]

@RomainGoncalves Nice, few things: in docs instead of $f = FeedReader::read('https://news.google.com/news/rss', null, $options); it should be $f = FeedReader::read('https://news.google.com/news/rss', 'default', $options); otherwise it will not load configuration.

I think it will be more flexible if to overwrite all options with passed to read() options instead of only curl options. It will be more flexible, the difference will be $f = FeedReader::read('https://news.google.com/news/rss', 'default', {'curl_options' => $curlOptions});. Also it will allow to add curl_options to config file.

It will require a bit more edits to pass $options array together with $configuration and overwrite options in readConfig() function. I can do this edits a bit later.

[RomainGoncalves]

@vedmant Thanks for the nice words.
I'll be honest, I did this change because it's something that I need in my project right now. I did not anticipate to change all the options.
Would you be able to work on that on a subsequent PR and let this one be merged?
Also, I will work on the fact that users would have to write the default configuration.

[vedmant]

@RomainGoncalves Ok, I'll merge current PR, but I'll change it a bit later, to avoid breaking change could you just update to make it pass curl options as 'curl_options' key in array: $f = FeedReader::read('https://news.google.com/news/rss', 'default', {'curl_options' => $curlOptions});

[vedmant]

@RomainGoncalves Please fix ErrorException: Undefined index: curl_options, should be if (isset($options['curl_options'])) {

[RomainGoncalves]

@vedmant sorry about that. Somehow the tests did not catch on it. Fixed now

[RomainGoncalves]

@vedmant quick question, is that change sent to packagist straight away? Can I update my dependencies and have the changes now?
Thanks

[vedmant]

Yes, it should update now

…

On Fri, May 28, 2021, 9:28 PM Romain Goncalves ***@***.***> wrote: @vedmant <https://github.com/vedmant> quick question, is that change sent to packagist straight away? Can I update my dependencies and have the changes now? Thanks — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#14 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ABGRP5VDQTXA46R6W73WUQTTP7OERANCNFSM45UP4KMQ> .

[RomainGoncalves]

Thanks I saw that.