Fix issue #4501 Exception noise from OAuth and REST (API2 ) (#4642)

* Fix issue #4501 Exception noise from OAuth and REST (API2 )

* Update app/code/core/Mage/Api2/Model/Resource.php

* Update app/code/core/Mage/Api2/Exception.php

Co-authored-by: Sven Reichel <[email protected]>

* Update app/code/core/Mage/Api2/Exception.php

Co-authored-by: Sven Reichel <[email protected]>

* Update app/code/core/Mage/Api2/Exception.php

Co-authored-by: Sven Reichel <[email protected]>

* Update app/code/core/Mage/Api2/Exception.php

---------

Co-authored-by: Sven Reichel <[email protected]>

Author: kiatng

app/code/core/Mage/Api2/Exception.php

@@ -14,18 +14,33 @@
  */
 class Mage_Api2_Exception extends Exception
 {
+    /**
+     * Log the exception in the log file?
+     */
+    protected bool $shouldLog = true;
+
     /**
      * Exception constructor
      *
      * @param string $message
      * @param int $code
+     * @param bool $shouldLog
      */
-    public function __construct($message, $code)
+    public function __construct($message, $code, $shouldLog = true)
     {
         if ($code <= 100 || $code >= 599) {
             throw new Exception(sprintf('Invalid Exception code "%d"', $code));
         }
 
+        $this->shouldLog = $shouldLog;
         parent::__construct($message, $code);
     }
+
+    /**
+     * Check if exception should be logged
+     */
+    public function shouldLog(): bool
+    {
+        return $this->shouldLog;
+    }
 }

app/code/core/Mage/Api2/Model/Resource.php

@@ -601,9 +601,10 @@ protected function _render($data)
      *
      * @param string $message
      * @param int $code
+     * @param bool $shouldLog Log the error in the log file?
      * @throws Mage_Api2_Exception
      */
-    protected function _critical($message, $code = null)
+    protected function _critical($message, $code = null, $shouldLog = true)
     {
         if ($code === null) {
             $errors = $this->_getCriticalErrors();
@@ -615,7 +616,10 @@ protected function _critical($message, $code = null)
             }
             $code = $errors[$message];
         }
-        throw new Mage_Api2_Exception($message, $code);
+
+        Mage::dispatchEvent('api2_resource_critical', ['resource' => $this, 'message' => $message, 'code' => $code]);
+
+        throw new Mage_Api2_Exception($message, $code, $shouldLog);
     }
 
     /**

app/code/core/Mage/Api2/Model/Server.php

@@ -93,6 +93,11 @@ public function run()
             if ($response->isException()) {
                 throw new Mage_Api2_Exception('Unhandled simple errors.', self::HTTP_INTERNAL_ERROR);
             }
+        } catch (Mage_Api2_Exception $e) {
+            if ($e->shouldLog()) {
+                Mage::logException($e);
+            }
+            $this->_renderException($e, $renderer, $response);
         } catch (Exception $e) {
             Mage::logException($e);
             $this->_renderException($e, $renderer, $response);