Fix PerDbnameStatsdDataFactory metric prefix again
authoraddshore <addshorewiki@gmail.com>
Wed, 1 Aug 2018 21:22:13 +0000 (22:22 +0100)
committeraddshore <addshorewiki@gmail.com>
Wed, 1 Aug 2018 21:22:13 +0000 (22:22 +0100)
prefix = 'MediaWiki.' will result in 'MediaWiki..dbname'

Preventing that is the only purpose of the rtrim(), we don't support
whitespace there

Change-Id: I72383aae908dbc89688cd3bebba36d389cde3f29

includes/ServiceWiring.php

index 66b4c0c..b2211ec 100644 (file)
@@ -137,7 +137,7 @@ return [
                $config = $services->getMainConfig();
                $wiki = $config->get( 'DBname' );
                return new BufferingStatsdDataFactory(
-                       rtrim( $services->getMainConfig()->get( 'StatsdMetricPrefix' ) ) . '.' . $wiki
+                       rtrim( $services->getMainConfig()->get( 'StatsdMetricPrefix' ), '.' ) . '.' . $wiki
                );
        },