wikimedia/mediawiki-extensions-VisualEditor
1
0
Fork 0
mirror of https://gerrit.wikimedia.org/r/mediawiki/extensions/VisualEditor synced 2024-12-04 18:58:37 +00:00
Code Issues Packages Projects Releases Wiki Activity

Merge "Try using structured logging again"

Browse source
This commit is contained in:
jenkins-bot 2019-10-22 12:39:41 +00:00 committed by Gerrit Code Review
parent 0f1281750e f7ee7dc807
commit 182b9c78ad
2 changed files with 18 additions and 8 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

20
includes/ApiVisualEditor.php
View file

@ -134,13 +134,15 @@ class ApiVisualEditor extends ApiBase {
);
} else {
// error null, code not 200
$trace = ( new Exception )->getTraceAsString();
$this->logger->warning(
__METHOD__ . ": Received HTTP {$response['code']} from RESTBase for $path." .
" Trace: {$trace}" .
" Response: {$response['body']}" .
/** @phan-suppress-next-line PhanTypeInvalidDimOffset */
" Request If-Match: {$reqheaders['If-Match']}"
__METHOD__ . ": Received HTTP {code} from RESTBase",
[
'code' => $response['code'],
'trace' => ( new Exception )->getTraceAsString(),
'response' => $response['body'],
'requestPath' => $path,
'requestIfMatch' => $reqheaders['If-Match'] ?? '',
]
);
$this->dieWithError(
[ 'apierror-visualeditor-docserver-http', $response['code'] ],
@ -634,7 +636,11 @@ class ApiVisualEditor extends ApiBase {
$badetag = $params['badetag'];
$goodetag = $result['etag'] ?? '';
$this->logger->info(
__METHOD__ . ": Client reported bad ETag: $badetag, expected: $goodetag"
__METHOD__ . ": Client reported bad ETag: {badetag}, expected: {goodetag}",
[
'badetag' => $badetag,
'goodetag' => $goodetag,
]
);
}
if ( $params['paction'] === 'parse' ||

6
includes/ApiVisualEditorEdit.php
View file

@ -288,7 +288,11 @@ class ApiVisualEditorEdit extends ApiVisualEditor {
"?$
/x', $etag ) ) {
$this->logger->info(
__METHOD__ . ": Received funny ETag from client: $etag for $path"
__METHOD__ . ": Received funny ETag from client: {etag}",
[
'etag' => $etag,
'requestPath' => $path,
]
);
}
return $this->requestRestbase(