php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login
Request #72049 Implement CURLINFO_APPCONNECT_TIME into curl_getinfo
Submitted: 2016-04-18 14:17 UTC Modified: 2016-04-24 00:35 UTC
From: marcvdmeulen at gmail dot com Assigned: pierrick (profile)
Status: Closed Package: Documentation problem
PHP Version: Irrelevant OS:
Private report: No CVE-ID: None
View Add Comment Developer Edit
Welcome! If you don't have a Git account, you can't do anything here.
You can add a comment by following this link or if you reported this bug, you can edit this bug over here.
(description)
Block user comment
Status: Assign to:
Package:
Bug Type:
Summary:
From: marcvdmeulen at gmail dot com
New email:
PHP Version: OS:

 

 [2016-04-18 14:17 UTC] marcvdmeulen at gmail dot com
Description:
------------
Curl allows to give some extra information about the ssl/tls connection time with CURLINFO_APPCONNECT_TIME. This info is not returned yet when curl_getinfo is called without any options.

Here you can find the registered global
https://github.com/php/php-src/blob/e015faaf8f022f6338c0254ef2fd617f29c94c84/ext/curl/interface.c#L1069

and i am guessing it should be implemented in this block
https://github.com/php/php-src/blob/e015faaf8f022f6338c0254ef2fd617f29c94c84/ext/curl/interface.c#L3020

Test script:
---------------
<?php

$ch = curl_init('http://www.yahoo.com/');


curl_exec($ch);

if(!curl_errno($ch)){
    $info = curl_getinfo($ch);
}

curl_close($ch);

Expected result:
----------------
An array with the "appconnect_time" key and its value

Actual result:
--------------
The only way now is to get the value with 
<?php
curl_getinfo($ch,CURLINFO_APPCONNECT_TIME);
?>

which is not even documented.

Patches

Add a Patch

Pull Requests

Add a Pull Request

History

AllCommentsChangesGit/SVN commitsRelated reports
 [2016-04-24 00:35 UTC] pierrick@php.net
-Status: Open +Status: Closed -Package: cURL related +Package: Documentation problem -Assigned To: +Assigned To: pierrick
 [2016-04-24 00:35 UTC] pierrick@php.net
Not all variables need to be in the curl_getinfo response without any options.
But that's true that this option was not specified in the documentation.

The documentation is committed and should be online soon to reflect this new option.

Thanks
 
PHP Copyright © 2001-2024 The PHP Group
All rights reserved.
Last updated: Fri Apr 19 14:01:30 2024 UTC