WordPress.org

Make WordPress Core

Changes between Version 1 and Version 2 of Ticket #36320, comment 22


Ignore:
Timestamp:
03/30/2016 03:38:27 PM (6 years ago)
Author:
reidbusi
Comment:

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #36320, comment 22

    v1 v2  
    77Also, the defines for the values of CURLOPT_SSLVERSION are missing on my hosts setup. The integer values must be used.
    88
    9 wp_cron is related, because when you test the paypal ipn connection, wordpress may decide to run cron which it appears it does through curl, and when you look at the contents of the $url variable it contains the cron url (scope issue?). It will do this inconsistently (not on every request). This was revealed in my testing where I was dumping variables with print_r() to observe what was going on. If I have time later I can dig out the code I was using to test that and demonstrate it for you. Right now I am working on site content for a client and don't have time to repeat this at the moment.
     9wp_cron is related, because when you test the paypal ipn connection, wordpress may decide to run cron which it appears it does through curl, and when you look at the contents of the $url variable it contains the cron url (scope issue?). It will do this inconsistently (not on every request). Thus the need to use curl_getinfo() to determine the url currently being requested by curl. This was revealed in my testing where I was dumping variables with print_r() to observe what was going on. If I have time later I can dig out the code I was using to test that and demonstrate it for you. Right now I am working on site content for a client and don't have time to repeat this at the moment.
    1010
    1111So while we are making progress here, the current commit to the woo master branch is still not quite there in my view.