Skip to content

no error checking for curl_exec() #3

Description

@battika

Hello Maxime,

I spent some days with getting sfMelodyPlugin work in my Windows development environment and one thing I found was in sfOAuth.class.php there was no error checking implemented for curl_exec() so there was no indication whether curl had run okay or it had failed with some problem.

What I mean is, instead of using

$response = curl_exec($ci);

it might be better idea to implement some kind of error checking like:

if ( ! $response = curl_exec($ci))
{
 // Log an error here using the output of curl_error($ci);
}

I contributed a possible way of logging the error here if you want to have a look: https://github.com/battika/OAuthBase

Thank you for your great job in the Symfony OAuth area, it's very much appreciated.

Activity

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

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions