deputy 0.1.24 → 0.1.25

data/VERSION

@@ -1 +1 @@
-0.1.24
+0.1.25

data/deputy.gemspec

@@ -5,7 +5,7 @@
 
 Gem::Specification.new do |s|
   s.name = %q{deputy}
-  s.version = "0.1.24"
+  s.version = "0.1.25"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["Michael Grosser"]

data/lib/deputy.rb

@@ -114,8 +114,10 @@ module Deputy
   end
 
   def self.send_report(metric, value)
-    raise "Use with dot e.g. Mysql.status" if metric.split('.',2).size != 2
-    get "/report/#{CGI.escape metric}?value=#{CGI.escape value.to_s}"
+    # TODO sheriff does not accept anything else, fix it...
+    metric = metric.split('.',2)
+    metric = metric.map{|m| m.gsub(/[^-_a-zA-Z]/, '_') } * '.'
+    get "/report/#{metric}?value=#{CGI.escape value.to_s}"
   end
 
   def self.get(path)

data/spec/deputy_spec.rb

@@ -155,5 +155,11 @@ describe Deputy do
       FakeWeb.register_uri(:get, "http://sheri.ff/report/Xxx.yyy?value=123", :body => 'OK')
       Deputy.send_report('Xxx.yyy', '123').should == 'OK'
     end
+
+    it "escapes metric names" do
+      Deputy.stub!(:sheriff_url).and_return 'http://sheri.ff'
+      FakeWeb.register_uri(:get, "http://sheri.ff/report/X_xx.yy___y?value=123", :body => 'OK')
+      Deputy.send_report('X/xx.yy/./y', '123').should == 'OK'
+    end
   end
 end

metadata

@@ -5,8 +5,8 @@ version: !ruby/object:Gem::Version
   segments: 
   - 0
   - 1
-  - 24
-  version: 0.1.24
+  - 25
+  version: 0.1.25
 platform: ruby
 authors: 
 - Michael Grosser