Problems with looping through array containing cURL options and setting them

Question

I'm making a library that allows a user to dynamically add/remove cURL options before a request is made. The foreach loop looks like this:

$ch = curl_init($url);

// Cycle through each option and set them
foreach($setup['curl_options'] as $option => $value)
{
    echo '<p>' . $option . ' = ' . $value . '</p>';
    curl_setopt($ch, $option, $value);
}

The array key/values display correctly in the paragraphs, however when it comes to adding the values to the curl_setopt, I get the error:

curl_setopt() expects parameter 2 to be long, string given.

What am I doing wrong here?

Marc B · Accepted Answer · 2011-02-28 16:50:00Z

1

curl_setopt($ch, $option, $value);

Let's say $option is 'CURLOPT_HEADER'. Your code boils down to:

curl_setopt($ch, 'CURLOPT_HEADER', $value);

the name of the constant is now a string, not an actual constant. What you need to do is store the value of what the constant represents in your array when you build it:

$setup['curl_options'][] = array('CURLOPT_HEADER', true); // wrong
$setup['curl_options'][] = array(CURLOPT_HEADER, true); // right

answered Feb 28, 2011 at 16:50

Marc B

362k44 gold badges433 silver badges508 bronze badges

Sign up to request clarification or add additional context in comments.

3 Comments

soren.qvist Over a year ago

This seems to work :) However a funny thing pops up now when I loop the key/value pairs through paragraphs. If I loop through: CURLOPT_SSL_VERIFYPEER => FALSE and CURLOPT_SSL_VERIFYHOST => FALSE, the $option parameter now contains numbers? 64 for HOST and 81 for PEER ?

Marc B Over a year ago

That's the raw values of those constants. somewhere inside the php curl library, there's what amounts to define('CURLOPT_SSL_VERIFYPEER', 81).

Marc B Over a year ago

That's all those constant "strings" are - just a mapping from a friendly semi-readable string to the numbers that curl uses internally.

hsz · Accepted Answer · 2011-02-28 16:42:43Z

1

Second parameter (your $option) have to be a constant value.

Here http://php.net/manual/en/function.curl-setopt.php you have definition of all available constans.

If you already use that values of constans, you should parse it to (long) type.

If you just have names of that constans, use constant($option) but be sure that values are also uppercase and correct.

edited Feb 28, 2011 at 16:42

answered Feb 28, 2011 at 16:37

hsz

153k63 gold badges269 silver badges320 bronze badges

2 Comments

Marc B Over a year ago

However, constant() simply returns the value of a constant. problem is that he's passing a string into the second argument $option, not the constant name itself.

hsz Over a year ago

Exacly. He should pass constant's value. When you use curl_setopt($ch, CURLOPT_AUTOREFERER, true), you pass to function a value that is hidden in called constant. You can also look into PHP source code and check what value it is: define ('CURLOPT_AUTOREFERER', 58); and pass to the function just 58 instead of constant.

soju · Accepted Answer · 2011-02-28 17:08:47Z

0

Try with

curl_setopt($ch, constant($option), $value);

edited Feb 28, 2011 at 17:08

answered Feb 28, 2011 at 16:36

soju

25.4k3 gold badges70 silver badges71 bronze badges

1 Comment

hsz Over a year ago

eval is evil. There is in PHP dedicated function for that operation called constant().

Collectives™ on Stack Overflow

Problems with looping through array containing cURL options and setting them

3 Answers 3

3 Comments

2 Comments

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

3 Comments

2 Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related