search

Pierre-Lannoy / wp-decalog

Capture and log events, metrics and traces on your site. Make WordPress observable - finally!
https://perfops.one/
GNU General Public License v3.0
64 stars 8 forks source link

Remove curl calls #1

Closed Pierre-Lannoy closed 5 years ago

Pierre-Lannoy commented 5 years ago

To adhere to WP security rules it's needed (but not enforced) to use wp_remote_* function instead of direct curl calls. Currently Monolog go through curl, so there's something to change ;)

Pierre-Lannoy commented 5 years ago

I've just sent a mail to the WP Plugin Directory team to clarify their position on this point regarding "external libraries".

Pierre-Lannoy commented 5 years ago

Wow! What a responsive team. They send me an answer just now. To summarize:

You're allowed to include the Monolog library. We only require you to NOT use Curl if it's in your own code, not in libraries.

So, there's no reason to rewrite cURL calls. Hell yeah!