Gergő Tisza has uploaded a new change for review.
https://gerrit.wikimedia.org/r/213527
Change subject: Fix Sentry DSN whitespace issues
......................................................................
Fix Sentry DSN whitespace issues
file_get_contents adds a newline, which breaks raven-php.
Bug: T85188
Change-Id: Ia91eb179ea19b26955439a92d80fc12347909daf
---
M puppet/modules/role/manifests/sentry.pp
1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/vagrant
refs/changes/27/213527/1
diff --git a/puppet/modules/role/manifests/sentry.pp
b/puppet/modules/role/manifests/sentry.pp
index 78a71cc..6405470 100644
--- a/puppet/modules/role/manifests/sentry.pp
+++ b/puppet/modules/role/manifests/sentry.pp
@@ -14,7 +14,7 @@
settings => [
# OMG that's ugly. Did not find a better way to reuse
# the output of a script.
- "\$wgSentryDsn = file_get_contents('${dsn}');",
+ "\$wgSentryDsn = trim(file_get_contents('${dsn}'));",
],
}
--
To view, visit https://gerrit.wikimedia.org/r/213527
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ia91eb179ea19b26955439a92d80fc12347909daf
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/vagrant
Gerrit-Branch: master
Gerrit-Owner: Gergő Tisza <gti...@wikimedia.org>
_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
