-
-
Notifications
You must be signed in to change notification settings - Fork 2.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
sendHttpRequest() erroneously returning "1" #1393
Comments
(In [2239]) fixes #1393 - see http://bugs.php.net/39034; we now check for boolean true and set the response to an empty string to avoid the cast, i.e., (string)true == "1" |
According to man page for curl_setopt_array(), it'll return false immediately (ignoring future options) if it encounters an option that it can't set (from the options array parameter). |
(In [2242]) fixes #1393 - we now setopt() CURLOPT_FILE or CURLOPT_RETURNTRANSFER separately in case curl_setopt_array() failed early; we also wrap ob_start() ... ob_end_clean() around curl_exec() to protect against output leaking to browser for some other reason |
(In [2243]) refs #1393 - I dug deeper into the libcurl source and change history, and as a result, I'm reordering the curl options based on level of support in libcurl, i.e., oldest to latest version, increasing the likelihood that all settable options are set; added some comments |
(In [2244]) refs #1393 - and this is based on a review of ext/curl; the workaround (not yet implemented) is to parse the Location: headers manually |
Anthon, can you confirm you tested the code when a new release is published and that it displays the version number properly + downloads the files? thanks |
nod |
I updated Piwik to version 0.6.4, and I still have this bug. In the top and to the right of the screen, it's written: New Update: Piwik 1 and if I move the cursor over: Piwik 1 est maintenant disponible. |
jpfle: this was fixed in 0.6.3; if you upgraded from an earlier version, then you'll have to wait up to 8 hrs for your site to re-ping our server. You can reset it manually, e.g.,
Then refresh your dashboard. If that doesn't work, then send me an email (anthon at piwik.org) with your phpinfo(). |
Replying to vipsoft:
You're right. That was the problem. Thanks a lot. |
Three forum posters and one email to [email protected] all reporting that the Piwik dashboard now says the current version available for download is "1".
The text was updated successfully, but these errors were encountered: