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

    XMLWordPrintable

    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:

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

      [[email protected] 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 Unassigned
              Reporter:
              lvlint67 daniel widrick
              Votes:
              2 Vote for this issue
              Watchers:
              9 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved: