[RESOLVED] cUrl, headers and CURLOPT_HEADERFUNCTION

neerav

I have the following code. do_curl() has been working and still works properly. I recently tried to add the ability to do something with the headers. Thus, I added the CURLOPT_HEADERFUNCTION and _setheader() function based on the php.net/curl_setopt user comments for using this within an object.

_setheader sees each header, the if...elseif works, as well as the $matches in the preg_match. However, $this->attach_info still remains empty when the print_r is uncommented. Using the example in the manual, $this->headers remained empty too. Using php 4.4.4. Though the example author uses php5, there is not reason why it would not work under php4.

I have spent hours trying to figure out what seemingly is a simple matter and could use some fresh eyeballs. Can someone figure out where I have erred?

function do_curl($some, $variables_which_are, $used_in_the_function) {

$this->attach_info = array();
$fp = fopen($filename, "wb");
if ($fp) {
	$c = curl_init();
	curl_setopt($c, CURLOPT_FILE, $fp);
	curl_setopt($c, CURLOPT_COOKIE, $cookie_string);
	curl_setopt($c, CURLOPT_URL, $query);
	curl_setopt($c, CURLOPT_SSL_VERIFYHOST, 2);	 
	curl_setopt($c, CURLOPT_SSL_VERIFYPEER, FALSE);
	curl_setopt($c, CURLOPT_USERAGENT, $user_agent);
	curl_setopt($c, CURLOPT_REFERER, $referrer);
	curl_setopt($c, CURLOPT_HEADERFUNCTION, array($this,'_setHeader'));

	curl_exec($c);
	curl_close($c);
	fclose($fp);
	//print_r($this,true));
}
}

function _setHeader($ch, $header) {
	if (preg_match("/Content-Length: (\d*)/",$header,$matches)) {
		$this->attach_info['size'] = $matches[1];
	} elseif (preg_match("/Content-Disposition:.*?filename=\"(.*)\"/",$header,$matches)) {
		$this->attach_info['filename'] = $matches[1];
	} elseif (preg_match("/Content-Type: ([^; ]*?)/",$header,$matches)) {
		$this->attach_info['type'] = $matches[1];
	}
	return strlen($header);
}

neerav

After about 10 days, I'm wondering if anyone might have any ideas since my further searches elsewhere have not yielded any insight? Did I leave out important information in my post?

Installer

When you give print_r() a second argument of true, the value is returned instead of directly output. So you would have to assign it to a variable, then echo that variable. That seems to be why you get no output. As for $this->headers, I don't see anywhere that you assign anything to it, such as $this->headers[] = $header;. Also, your preg_match() for the content length doesn't work for me.

Otherwise, after correcting those things, it works.

neerav

I was actually print_r'ing to a debugging file. Should have left out the "true" in my code snippet. Even then, $this had all of the other data except $this->attach_info Sorry, that just confounded my question.

The preg_match for Content length was working for me, when echoing the match inside the if{}. Maybe it needed to be case insensitive for the server you were testing? I'll add case-insensitivity, just to be on the safe side.

$this->headers was in the manual example. I was using $this->attach_info. I apologize if I was not clear.

After spending more hours pulling my hair out, I have found the problem. This line:

curl_setopt($c, CURLOPT_HEADERFUNCTION, array($this,'_setHeader'));

should be:

curl_setopt($c, CURLOPT_HEADERFUNCTION, array(&$this,'_setHeader'));

The "&" was all important. On hindsight, it makes sense. But two weeks for the want of an "&" :bemused: The next thought is why this is not seen in any of the examples anywhere, so why does it work for others without it and not for me? 😕

Thank you, Installer, for your assistance. Very much appreciated. Until the next time we meet... all the best!

Weedpacket

neerav wrote:
I was actually print_r'ing to a debugging file. Should have left out the "true" in my code snippet.

Yes; with the "true" there the data was being returned so that you could put it in a variable. But you weren't putting it in a variable, just ignoring it, so it ended up being blown out the back of your PC by the cooling fan. It certainly wasn't being written to any debugging file.

neerav

I actually use another function to print to the debugging file, and replaced it with print_r and it's incorrect usage when I posted the code sample above. Trust me, I WAS seeing the rest of object $this EXCEPT for what was supposed to be appended in setHeader(). Removing the True and having a simple print_r to print to screen, the object was still losing the results of setHeader()

I'm sorry for the confusion.

For anyone else using CURLOPT_HEADERFUNCTION in an object and having a problem (and using php 4.4.4?), just remember to pass the object $this as &$this