EBernhardson has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/295765

Change subject: Match the expected format of 'response' log key
......................................................................

Match the expected format of 'response' log key

All sources that log a 'response' value need to use the same format.
Make EventBus compatible with how Math extension utilizes the response
field.

Bug: T138539
Change-Id: I71bd2ee45c132d698fd8689d86674497bb3fa817
---
M EventBus.php
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/EventBus 
refs/changes/65/295765/1

diff --git a/EventBus.php b/EventBus.php
index d8c8354..1b5fd29 100644
--- a/EventBus.php
+++ b/EventBus.php
@@ -70,7 +70,7 @@
 
        private function onError( $ret ) {
                $message = empty( $ret['error'] ) ? $ret['code'] . ': ' . 
$ret['reason'] : $ret['error'];
-               $context = [ 'response' => $ret['body'] ];
+               $context = [ 'response' => [ 'body' => $ret['body'] ] ];
 
                $logger = LoggerFactory::getInstance( 'EventBus' );
                $logger->error( "Unable to deliver event: ${message}", $context 
);

-- 
To view, visit https://gerrit.wikimedia.org/r/295765
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I71bd2ee45c132d698fd8689d86674497bb3fa817
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/EventBus
Gerrit-Branch: wmf/1.28.0-wmf.3
Gerrit-Owner: EBernhardson <ebernhard...@wikimedia.org>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to