Skip to content

Commit f13db70

Browse files
deer-wmdedati18
andauthored
logging: adjust service name by environment (#514)
* adjust logging service name via env var * Update dist-persist/wbstack/src/Logging/CustomLogger.php Co-authored-by: Dat WMDE <[email protected]> * add environment to context array --------- Co-authored-by: Dat WMDE <[email protected]>
1 parent 67b4e44 commit f13db70

File tree

2 files changed

+18
-4
lines changed

2 files changed

+18
-4
lines changed

dist-persist/wbstack/src/Logging/CustomLogger.php

Lines changed: 9 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -61,13 +61,20 @@ private function doLog( $level, $message, $context ) {
6161
$payload[ '@type' ] = 'type.googleapis.com/google.devtools.clouderrorreporting.v1beta1.ReportedErrorEvent';
6262
}
6363

64+
$envString = '';
65+
if ($env = getenv('MW_ENVIRONMENT')) {
66+
$envString = " ({$env})";
67+
}
68+
6469
$payload[ 'severity' ] = $level;
6570
$payload[ 'serviceContext' ] = [
66-
'service' => 'WBaaS MediaWiki',
71+
'service' => 'WBaaS MediaWiki' . $envString,
6772
'version' => MW_VERSION,
6873
];
6974
$payload[ 'context' ] = [
70-
'request_uri' => $_SERVER['REQUEST_URI'] ?? '',
75+
'environment' => $env ?? '',
76+
77+
'request_uri' => $_SERVER['REQUEST_URI'] ?? '',
7178

7279
// set in /includes/Defines.php
7380
'mediawiki' => MW_VERSION,

dist/wbstack/src/Logging/CustomLogger.php

Lines changed: 9 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -61,13 +61,20 @@ private function doLog( $level, $message, $context ) {
6161
$payload[ '@type' ] = 'type.googleapis.com/google.devtools.clouderrorreporting.v1beta1.ReportedErrorEvent';
6262
}
6363

64+
$envString = '';
65+
if ($env = getenv('MW_ENVIRONMENT')) {
66+
$envString = " ({$env})";
67+
}
68+
6469
$payload[ 'severity' ] = $level;
6570
$payload[ 'serviceContext' ] = [
66-
'service' => 'WBaaS MediaWiki',
71+
'service' => 'WBaaS MediaWiki' . $envString,
6772
'version' => MW_VERSION,
6873
];
6974
$payload[ 'context' ] = [
70-
'request_uri' => $_SERVER['REQUEST_URI'] ?? '',
75+
'environment' => $env ?? '',
76+
77+
'request_uri' => $_SERVER['REQUEST_URI'] ?? '',
7178

7279
// set in /includes/Defines.php
7380
'mediawiki' => MW_VERSION,

0 commit comments

Comments
 (0)