deputy 0.1.25 → 0.1.26

data/VERSION

@@ -1 +1 @@
-0.1.25
+0.1.26

data/deputy.gemspec

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

data/lib/deputy.rb

@@ -114,10 +114,8 @@ module Deputy
   end
 
   def self.send_report(metric, value)
-    # 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}"
+    raise "separate #{metric} with a ." unless metric.split('.',2).size == 2
+    get "/report/#{CGI.escape metric}?value=#{CGI.escape value.to_s}"
   end
 
   def self.get(path)

data/spec/deputy_spec.rb

@@ -158,8 +158,8 @@ describe Deputy do
 
     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'
+      FakeWeb.register_uri(:get, "http://sheri.ff/report/Xxx.yy%3Fy?value=123", :body => 'OK')
+      Deputy.send_report('Xxx.yy?y', '123').should == 'OK'
     end
   end
 end

metadata

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