Details

    • Type: Bug Bug
    • Status: Open
    • Priority: High High
    • Resolution: Unresolved
    • Affects Version/s: 2013.09
    • Fix Version/s: None
    • Component/s: Caching
    • Labels:

      Description

      I use Piwik as web analytics application on my site. This application create a cookie key with a * character. When eZ Publish save this Cookie in a stash cache file, the site is broken.

      example in phperror.log

      PHP Parse error: syntax error, unexpected '*' in /ezpublish/cache/prod/stash/0fea6a13c52b4d47/25368f24b045ca84/ea89d5a5fa57ab5b/b4f9262a31f00ac7.php on line 2

      b4f9262a31f00ac7.php

      <?php
      /* Cachekey: cache/ez_user_hash/_pk_id.190.52a7=67f08ff5ed0cf27d.1384950943.2.1385022003.1384952298.; ezsessid86af0f3a92ebf2d522cb0a6948f16d27=0dd9b3f52482a8884faffefaeb395e90; is_logged_in=true; _pk_cvar.190.52a7=%7b%221%22%3a%5b%22user%22%2c%22admin%22%5d%7d; _pk_ses.190.52a7=*/ */
      /* Type: array */
      $expiration = 1385022546;
      $data = array();

      /* Child Type: string */
      $data['return'] = "ec35701ab7f34b037ee156a397bfb2ed255acc0c";

      /* Child Type: integer */
      $data['createdOn'] = 1385022005;

        Activity

        Hide
        Gaetano Giunta (Inactive) added a comment - - edited

        Most likely a bug in Stash, reported upstream as https://github.com/tedivm/Stash/issues/111
        Offending code is vendor\tedivm\stash\src\Stash\Driver\FileSystem.php

        Show
        Gaetano Giunta (Inactive) added a comment - - edited Most likely a bug in Stash, reported upstream as https://github.com/tedivm/Stash/issues/111 Offending code is vendor\tedivm\stash\src\Stash\Driver\FileSystem.php
        Hide
        Sebastian Blank added a comment -

        Yesterday I created a pull request. Now my pull request has been merged.
        https://github.com/tedivm/Stash/commit/fc4f6f1010c33db736b9fde9bc8e65c136f76148

        Show
        Sebastian Blank added a comment - Yesterday I created a pull request. Now my pull request has been merged. https://github.com/tedivm/Stash/commit/fc4f6f1010c33db736b9fde9bc8e65c136f76148
        Hide
        Sebastian Blank added a comment -
        Show
        Sebastian Blank added a comment - With the Update to Stash 0.3.0 this bug is fixed. https://github.com/ezsystems/ezpublish-kernel/commit/deb41cb4698524d14ee36cdd433d679576a749d3

          People

          • Assignee:
            Unassigned
            Reporter:
            Sebastian Blank
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated: