Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP

Loading…

Make sure globalSuffix is undefined when not used. #32

Merged
merged 1 commit into from

2 participants

@ralphm

Otherwise, StatsD will create metrics like '.../metric/rate/.wsp'
instead of '.../metric/rate.wsp'.

This fixes the regression that ended up in 1.1.7. See #31.

@ralphm ralphm Make sure globalSuffix is undefined when not used.
Otherwise, StatsD will create metrics like '.../metric/rate/.wsp'
instead of '.../metric/rate.wsp'.
106994b
@hectcastro
Owner

Thanks, @ralphm! And apologies for introducing the regression.

@hectcastro hectcastro merged commit 476df18 into hectcastro:develop
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on May 12, 2014
  1. @ralphm

    Make sure globalSuffix is undefined when not used.

    ralphm authored
    Otherwise, StatsD will create metrics like '.../metric/rate/.wsp'
    instead of '.../metric/rate.wsp'.
This page is out of date. Refresh to see the latest.
Showing with 4 additions and 0 deletions.
  1. +4 −0 templates/default/config.js.erb
View
4 templates/default/config.js.erb
@@ -13,7 +13,11 @@
"graphite": {
"legacyNamespace": <%= @legacy_namespace %>,
"globalPrefix": "<%= @global_prefix %>",
+<% if !@global_suffix.empty? %>
"globalSuffix": "<%= @global_suffix %>",
+<% else %>
+ "globalSuffix": undefined,
+<% end %>
"prefixCounter": "<%= @prefix_counter %>",
"prefixTimer": "<%= @prefix_timer %>",
"prefixGauge": "<%= @prefix_gauge %>",
Something went wrong with that request. Please try again.