MM-13027: Don't include a diagnostics segment key in dev builds. (#2131)

mattermost · Dec 11, 2018 · 5ec394c · 5ec394c
1 parent fdf72ed
commit 5ec394c
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/components/root/root.jsx b/components/root/root.jsx
@@ -146,7 +146,7 @@ export default class Root extends React.Component {
  const diagnosticId = this.props.diagnosticId;
 
  /*eslint-disable */
- if (segmentKey != null && segmentKey !== '' && this.props.diagnosticsEnabled) {
+ if (segmentKey != null && segmentKey !== '' && !segmentKey.startsWith('placeholder') && this.props.diagnosticsEnabled) {
  !function(){var analytics=global.window.analytics=global.window.analytics||[];if(!analytics.initialize)if(analytics.invoked)window.console&&console.error&&console.error("Segment snippet included twice.");else{analytics.invoked=!0;analytics.methods=["trackSubmit","trackClick","trackLink","trackForm","pageview","identify","group","track","ready","alias","page","once","off","on"];analytics.factory=function(t){return function(...args){var e=Array.prototype.slice.call(args);e.unshift(t);analytics.push(e);return analytics}};for(var t=0;t<analytics.methods.length;t++){var e=analytics.methods[t];analytics[e]=analytics.factory(e)}analytics.load=function(t){var e=document.createElement("script");e.type="text/javascript";e.async=!0;e.src=("https:"===document.location.protocol ? "https://":"http:https://")+"cdn.segment.com/analytics.js/v1/"+t+"/analytics.min.js";var n=document.getElementsByTagName("script")[0];n.parentNode.insertBefore(e,n)};analytics.SNIPPET_VERSION="3.0.1";
  analytics.load(segmentKey);
 

diff --git a/utils/constants.jsx b/utils/constants.jsx
@@ -1197,7 +1197,7 @@ export const Constants = {
  AUTOCOMPLETE_TIMEOUT: 100,
  ANIMATION_TIMEOUT: 1000,
  SEARCH_TIMEOUT_MILLISECONDS: 100,
- DIAGNOSTICS_SEGMENT_KEY: 'fwb7VPbFeQ7SKp3wHm1RzFUuXZudqVok',
+ DIAGNOSTICS_SEGMENT_KEY: 'placeholder_segment_key',
  TEST_ID_COUNT: 0,
  CENTER: 'center',
  RHS: 'rhs',