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 Incident report
    • Status: Closed
    • Priority: Minor 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,

        Issue Links

          Activity

          Hide
          richlv added a comment -

          with the final php7 out yesterday (december 3rd), this issue might be worth re-evaluating

          Show
          richlv added a comment - with the final php7 out yesterday (december 3rd), this issue might be worth re-evaluating
          Hide
          vitalijs.cemeris added a comment - - edited

          (1) No translation strings changed

          Gunars Pujats CLOSED

          Show
          vitalijs.cemeris added a comment - - edited (1) No translation strings changed Gunars Pujats CLOSED
          Hide
          vitalijs.cemeris added a comment -

          RESOLVED in development branch svn://svn.zabbix.com/branches/dev/ZBX-9659

          Show
          vitalijs.cemeris added a comment - RESOLVED in development branch svn://svn.zabbix.com/branches/dev/ZBX-9659
          Hide
          Ricardo Felipe Klein added a comment -

          The web installer on 3.0.0rc1 still complains about this php setting.

          Show
          Ricardo Felipe Klein added a comment - The web installer on 3.0.0rc1 still complains about this php setting.
          Hide
          Billy D. added a comment -

          Hi, any idea when the update will be published?

          Show
          Billy D. added a comment - Hi, any idea when the update will be published?
          Hide
          Gunars Pujats added a comment -

          Successfully tested!

          Show
          Gunars Pujats added a comment - Successfully tested!
          Hide
          Ivo Kurzemnieks added a comment - - edited

          Subissue (1) still open.

          Show
          Ivo Kurzemnieks added a comment - - edited Subissue (1) still open.
          Hide
          vitalijs.cemeris added a comment - - edited

          Fixed in:

          • pre-3.0.2rc1 r58879
          • pre-3.1.0 r58880
          Show
          vitalijs.cemeris added a comment - - edited Fixed in: pre-3.0.2rc1 r58879 pre-3.1.0 r58880

            People

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

              Dates

              • Created:
                Updated:
                Resolved: