grape-appsignal 0.2.1 → 0.2.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: e6c121e22a45831cb724e73f4ed0d9d24f46f384
-  data.tar.gz: b12d4b6815e8b599f627f2bc2500f9cd8abddd68
+  metadata.gz: 7b3640f070cbad3d5c18fbd06cd7994528a319fa
+  data.tar.gz: 350802302c0d2482ab6d420e003147c2b33a6a6d
 SHA512:
-  metadata.gz: 6b12d3e0dc5c05a095f8755ecdddd806616659e2acf2dac27bf5f797b826caee0f4bb94d5fc7d97d0213f14ef31f79bc6774a7cdd717900c1cc34f1f0090d24b
-  data.tar.gz: c2bc9f2085902de1f6ff2e6a406db6b02baab1380f74ae60190fbfa27a9ae02866311b3f8b2a226d94d1b912b67ec45d5ab579d7856904b94d878005fecf52ce
+  metadata.gz: 17beb0ede9668ebce99d084551b9d46b43bb6295f0823f469d1cfb8558ad56fe7699b3e32bbe692e223f832ac25ccbcadc8bc5258081210f4f39fdf487483fb8
+  data.tar.gz: fdbdca8b4ecbdc8bd654c81b96256ac5ba8b24debf5ba8b4103f8a7a9e29407de0e8b14cd014c237fb7cd868f379b2d9c9e2dd5e9cb777798d586d4de109f959

data/lib/grape-appsignal/middleware.rb

@@ -13,9 +13,12 @@ module Appsignal
         method = env['REQUEST_METHOD']
 
         request_path = env['api.endpoint'].routes.first.route_path[1..-1].sub(/\(\.:format\)\z/, "")
+
         metric_name  = "grape.#{req.request_method}.#{request_path}"
-        action = "#{env['PATH_INFO']}"
+        metric_name = metric_name.gsub(/\/:?/, '.')
 
+        action = "#{env['PATH_INFO']}"
+        action = action.gsub(/\//, '::')
 
         ActiveSupport::Notifications.instrument(metric_name, { method: method, path: request_path, action: action, class: "API" } ) do |payload|
           @app.call(env)

data/lib/grape-appsignal/version.rb

@@ -1,5 +1,5 @@
 module Appsignal
   module Grape
-    VERSION = "0.2.1"
+    VERSION = "0.2.2"
   end
 end

data/spec/grape-appsignal/middleware_spec.rb

@@ -37,12 +37,12 @@ describe Appsignal::Grape::Middleware do
 
     it "delivers a payload consistent with the API call."do
       expect(subject.payload ).to eq(
-        { method: "GET" , path: "hello/:id", action: "/hello/1337", class: "API"}
+        { method: "GET" , path: "hello/:id", action: "::hello::1337", class: "API"}
       )
     end
 
     it "names the payload consistent with the API call."do
-      expect(subject.name ).to eq("grape.GET.hello/:id")
+      expect(subject.name ).to eq("grape.GET.hello.id")
     end
 
     context "verify the api request" do
@@ -70,12 +70,12 @@ describe Appsignal::Grape::Middleware do
 
     it "delivers a payload consistent with the API call."do
       expect(subject.payload ).to eq(
-        { method: "GET" , path: "api/:version/hello/:name", action: "/api/v1/hello/mark", class: "API"}
+        { method: "GET" , path: "api/:version/hello/:name", action: "::api::v1::hello::mark", class: "API"}
       )
     end
 
     it "names the payload consistent with the API call."do
-      expect(subject.name ).to eq("grape.GET.api/:version/hello/:name")
+      expect(subject.name ).to eq("grape.GET.api.version.hello.name")
     end
 
     context "verify the api request" do

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: grape-appsignal
 version: !ruby/object:Gem::Version
-  version: 0.2.1
+  version: 0.2.2
 platform: ruby
 authors:
 - Mark Madsen