The openmage-sentry from luigifab

openmage-sentry's People

Contributors

Stargazers

Watchers

openmage-sentry's Issues

"Sentry is not fully installed: $sentry not found!"

Are there any dependencies?

Trying to move from Firegento_Logger and I'm getting this error when enabling Sentry:
Sentry is not fully installed: $sentry not found!

Used the normal diff (not "more").
Installed via FTP but do not see any composer dependencies.

Exception.log logging:
isDeveloperMode:off sentry:off
before the exceptions (unrelated)

OpenMage 19.4.23
PHP 8.1

API errors not sent to Sentry

The following PHP code must be added:

diff --git a/web/app/code/core/Mage/Api/Model/Server/Handler/Abstract.php b/web/app/code/core/Mage/Api/Model/Server/Handler/Abstract.php
index 16f6d66155..ddf7f86bb7 100644
--- a/web/app/code/core/Mage/Api/Model/Server/Handler/Abstract.php
+++ b/web/app/code/core/Mage/Api/Model/Server/Handler/Abstract.php
@@ -35,9 +35,15 @@ abstract class Mage_Api_Model_Server_Handler_Abstract
      * @param string $errorFile
      * @return bool
      */
-    public function handlePhpError($errorCode, $errorMessage, $errorFile)
+    public function handlePhpError($errorCode, $errorMessage, $errorFile, $errLine, $context = [])
     {
-        Mage::log($errorMessage . $errorFile);
+        global $sentry;
+        if ($sentry) {
+            $sentry->handleError($errorCode, $errorMessage, $errorFile, $errLine, $context);
+        }
+        else {
+            Mage::log($errorMessage . '  in ' . $errorFile . ' on line ' . $errLine);
+        }
         if (in_array($errorCode, [E_ERROR, E_USER_ERROR, E_RECOVERABLE_ERROR])) {
             $this->_fault('internal');
         }

Recommend Projects