Skip to content

Commit a59ed06

Browse files
committed
Only log external request in error
1 parent d54cb00 commit a59ed06

File tree

3 files changed

+7
-3
lines changed

3 files changed

+7
-3
lines changed

Model/LogHandle.php

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -123,7 +123,11 @@ public function after(
123123
$this->lastLog->setResponseBody($resposeBody);
124124
$this->lastLog->setResponseCode($responseCode);
125125
$this->lastLog->setResponseDatetime($responseDateTime);
126-
$this->logResourceModel->save($this->lastLog);
126+
if ($responseCode === '200') {
127+
$this->logResourceModel->delete($this->lastLog);
128+
} else {
129+
$this->logResourceModel->save($this->lastLog);
130+
}
127131
} catch (Exception $exception) {
128132
$this->logger->error(__('Cant complete webapi log save because of error: %1', $exception->getMessage()));
129133
}

Plugin/FrontControllerDispatchAfter.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -54,7 +54,7 @@ public function __construct(
5454
*/
5555
public function afterDispatch(Rest $subject, $result, RequestInterface $request)
5656
{
57-
if ($this->config->isEnabled()) {
57+
if ($this->config->isEnabled() && !$request->isXmlHttpRequest()) {
5858
$exceptions = $result->getException();
5959

6060
if (!empty($exceptions)) {

Plugin/FrontControllerDispatchBefore.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,7 @@ public function __construct(
5353
*/
5454
public function beforeDispatch(Rest $subject, RequestInterface $request)
5555
{
56-
if ($this->config->isEnabled()) {
56+
if ($this->config->isEnabled() && !$request->isXmlHttpRequest()) {
5757
$requestMethod = $request->getMethod();
5858
$requestorIp = $request->getClientIp();
5959
$requestPath = $request->getUriString();

0 commit comments

Comments
 (0)