Uploaded image for project: 'Z-Push'
  1. Z-Push
  2. ZP-1506

LOGLEVEL_WBXML does not work with ZLog::SpecialLogUser()

    Details

    • Type: Bug
    • Status: QA
    • Priority: Medium
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 2.5.1, 2.5.1 beta 1
    • Component/s: None
    • Labels:
      None

      Description

      EGroupware currently updating from z-push 2.3.x to lastet 2.5.0 installed via composer (https://packagist.org/packages/egroupware/z-push-dev from our own GitHub fork of z-push. I'm more then happy to use an official z-push composer package, if there were one.)

      We call ZLog::SpecialLogUser() to get a log for a specific device and have LOGUSERLEVEL = LOGLEVEL_DEVICEID, to log the XML.

      This no longer works, as WBXML-decoder and -encoder call ZLog::IsWbxmlDebugEnabled(), which does not take the ZLog::SpecialLogUser() into account.

      Either ZLog::SpecialLogUser() must track and report it in ZLog::IsWbxmlDebugEnabled(), or the logger-backend eg.
      public function HasSpecialLogUsers()

      {     return !empty($this->specialLogUsers) || $this->isAuthUserInSpecialLogUsers; }

      Or we need an other way to enable a device-specific log.

      I'm happy to implement or fix it, thought I want to ask first how you want to have it done

      Ralf

        Gliffy Diagrams

          Attachments

            Activity

              People

              • Assignee:
                Unassigned
                Reporter:
                RalfBecker Ralf Becker
              • Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: