Logger AbstractAdapter throwing exception in v5.

[oyeaussie]

Hello All,

I upgraded from v4 to v5 and since the upgrade, I see this exception being thrown on every reply I receive from the server.

There is an active transaction in phalcon/Logger/Adapter/AbstractAdapter.zep on line 64

I am using transaction to log messages using begin() and commit()

        $streamAdapter = new Stream($savePath . 'exceptions.log');

        if ($this->logsConfig->exceptions) {
            $this->logExceptions = new PhalconLogger(
                'messages',
                ['exceptions'        => $streamAdapter]
            );

            $this->logExceptions->setLogLevel($this->logExceptions::ALERT);

            $this->logExceptions->getAdapter('exceptions')->begin();
        }

To commit:

                if ($this->logExceptions->getAdapter('exceptions')->inTransaction()) {
                    $this->logExceptions->getAdapter('exceptions')->commit();
                }

Comparing the AbstractAdapter.zep files from v4 to v5, I see this exception was added to the __destruct() instead of this->commit()

Any suggestion on how to fix this?

[oyeaussie]

Resolved!

I looked at the zep file from version 4 and version 5. As per a security concern that was raised a while back, it was decided to throw an execption instead of commit any uncommited transactions, forcing the developer to make sure they commit the logs, which clears the transaction queue correctly.

Thanks.