appsignal 2.1.1 → 2.1.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 9ef0883ffd29bd81de9c7a70b6112e3e8e877d21
-  data.tar.gz: 750cfc0639962b2f46434e6f28715acdb97db8c3
+  metadata.gz: e046fb025d367006e6b7917b5cbf63bfd592271d
+  data.tar.gz: a2cc0879ae2f6386ab860e17d78914428babd5a9
 SHA512:
-  metadata.gz: 4523a7ca1c3c4860643779073c86802ba7331038fa0633280509d1b4d82fc811fb77c5faaa96a672e700d0d6c7e786051d874e674b0d05bf8979238c875e26f5
-  data.tar.gz: ccb7f37840632b613902a3eda64fab7534b86e5147116869fc3729f560eccba6e2318d00bb3ab9a79fb1e2c22c48b3d2965dc711780b0d26b22ded9eaab089ca
+  metadata.gz: 1d99afd1e739b01f82e575c0237afea2a2c4cace49e2634b9c1965c16eaea09f4bac37d31c24bba2a4454cc70256922431cd11dd3828653ffddcc14214fea39c
+  data.tar.gz: aee0fbbb44af79823ca0cb68f5801e58d1302aa0e6158bfbe165887dd0db94ef83789ff265fe24a5c8e49808c33fc5718b60953460530fe476688321021fc9a0

data/CHANGELOG.md

@@ -1,3 +1,6 @@
+# 2.1.2
+* Fix error with Grape request methods defined with symbols. PR #259
+
 # 2.1.1
 * Fix DNS issue related to musl build.
   Commit 732c877de8faceabe8a977bf80a82a6a89065c4d

data/lib/appsignal/integrations/grape.rb

@@ -22,13 +22,13 @@ module Appsignal
           transaction.set_error(error)
           raise error
         ensure
-          request_method = request.request_method
+          request_method = request.request_method.to_s.upcase
           path = request.path # Path without namespaces
           endpoint = env["api.endpoint"]
 
           if endpoint && endpoint.options
             options = endpoint.options
-            request_method = options[:method].first
+            request_method = options[:method].first.to_s.upcase
             klass = options[:for]
             namespace = endpoint.namespace
             namespace = "" if namespace == "/"

data/lib/appsignal/version.rb

@@ -1,5 +1,5 @@
 require "yaml"
 
 module Appsignal
-  VERSION = "2.1.1"
+  VERSION = "2.1.2"
 end

data/spec/lib/appsignal/integrations/grape_spec.rb

@@ -98,6 +98,30 @@ if DependencyHelper.grape_present?
           end
         end
 
+        context "with route" do
+          let(:app) do
+            Class.new(::Grape::API) do
+              route [:get, :post], "hello" do
+                "Hello!"
+              end
+            end
+          end
+          let(:env) do
+            http_request_env_with_data \
+              "api.endpoint" => api_endpoint,
+              "REQUEST_METHOD" => "GET",
+              :path => ""
+          end
+
+          it "sets non-unique route path" do
+            expect(transaction).to receive(:set_action).with("GET::GrapeExample::Api#/hello")
+            expect(transaction).to receive(:set_metadata).with("path", "/hello")
+            expect(transaction).to receive(:set_metadata).with("method", "GET")
+          end
+
+          after { middleware.call(env) }
+        end
+
         context "with route_param" do
           let(:app) do
             Class.new(::Grape::API) do

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: appsignal
 version: !ruby/object:Gem::Version
-  version: 2.1.1
+  version: 2.1.2
 platform: ruby
 authors:
 - Robert Beekman
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-03-06 00:00:00.000000000 Z
+date: 2017-03-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rack