Uploaded image for project: 'ZABBIX BUGS AND ISSUES'
  1. ZABBIX BUGS AND ISSUES
  2. ZBX-9659

always_populate_raw_post_data in PHP7.0.0alpha2

    Details

    • Type: Incident report
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.0.17rc1, 2.2.12rc1, 2.4.5, 3.0.0alpha6
    • Fix Version/s: 3.0.2rc1, 3.2.0alpha1
    • Component/s: API (A), Frontend (F)
    • Labels:
    • Environment:
      Oracle Linux 7.1 (lxc container on OEL 7.1 Host - selinux disabled)
      php-7.0.0alpha2 (php-fpm)

      Description

      include/classes/setup/CFrontendSetup.php

      Appears to check PHP_VERSION >= 5.6 and when true checks the php.ini value of always_populate_raw_post_data

      This .ini value has been removed as of php-7.0.0alpha2 as per:
      https://github.com/php/php-src/blob/php-7.0.0alpha2/UPGRADING

      I'm not sure You care about the alpha versions of php yet but I figured I would attach the changes I made to get around the issue:

      ------------------------------------

      [root@zabbix htdocs]# git diff
      diff --git a/include/classes/setup/CFrontendSetup.php b/include/classes/setup/CFrontendSetup.php
      index babdaa7..393349f 100755
      
      --- a/include/classes/setup/CFrontendSetup.php
      +++ b/include/classes/setup/CFrontendSetup.php
      @@ -72,7 +72,7 @@ class CFrontendSetup {
                      }
       
                      // check for deprecated PHP 5.6.0 option 'always_populate_raw_post_data'
      -               if (version_compare(PHP_VERSION, '5.6', '>=')) {
      +               if (version_compare(PHP_VERSION, '5.6', '>=') && version_compare(PHP_VERSION, '7.0.0alpha2', '<')) {
                              $result[] = $this->checkPhpAlwaysPopulateRawPostData();
                      }
                      $result[] = $this->checkPhpSockets();
      

      -----------------------------------

      Thank you,

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                Unassigned
                Reporter:
                lvlint67 daniel widrick
              • Votes:
                2 Vote for this issue
                Watchers:
                9 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: