sapience 1.0.10 → 1.0.11

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: bd7f003c22fa41c883484585a8044f5d623884fd
-  data.tar.gz: d29f04a6137ec76d11e29250c0121404d79e50b2
+  metadata.gz: 3000f95011dbbb20f73e3cb9ba95f7df3270b84a
+  data.tar.gz: c079d45e7bd0aecb28c38e98fc656b8a38167ebe
 SHA512:
-  metadata.gz: 68a3f162e0a5e98f4147f597af4fecc041d711eb7400c470b26452da5da65c77eede3215a6861f130b0dbec6aa95516ee741a0a209675a6ecd9e68b0cf456672
-  data.tar.gz: 2a92fa93050032dd204d9d1a37f626424b867926ed099def4c48eec20c1ab3e7bf5c09b5158d22d947e5cc4ab1f4142f4f32e1af4b520fc4f8b569bdc006db84
+  metadata.gz: 1cb03d2e5cdfecbb017f61d9e93557dd5d53fbe1c7687a60758da8b0cbc14dc149ed3daa3ebd38040747316840f1622b1e8432eced86196bccf6e081b129276b
+  data.tar.gz: f5156e824985e613c78deb80b9f32ce14c83c4907cd81294cf86fbc428f19f7473b3d46b80fac296f9d7032727ad7da8ef3b30e8ad32882470b48a9c687a1eb8

data/CHANGELOG.md

@@ -1,3 +1,7 @@
+## v1.0.11
+
+- Fix a NoMethodError with logging the response format in grape
+
 ## v1.0.10
 
 - Automatically add default `datadog` appender when calling `Sapience.metrics` 

data/lib/sapience/extensions/grape/middleware/logging.rb

@@ -39,7 +39,7 @@ module Sapience
             {
               method: request.request_method,
               request_path: request.path,
-              format: env["api.endpoint"].instance_variable_get(:@app).instance_variable_get(:@app).options[:format],
+              format: response_format,
               status: response.try(:status) || 404,
               class_name: env["api.endpoint"].options[:for].to_s,
               action: "index",
@@ -60,6 +60,16 @@ module Sapience
 
           private
 
+          def content_type
+            request.env.fetch("CONTENT_TYPE") do
+              request.env["CONTENT-TYPE"]
+            end
+          end
+
+          def response_format
+            content_type.to_s.split("/").last
+          end
+
           def request
             @request ||= ::Rack::Request.new(env)
           end

data/lib/sapience/version.rb

@@ -1,3 +1,3 @@
 module Sapience
-  VERSION = "1.0.10"
+  VERSION = "1.0.11"
 end

data/test_apps/grape/.rspec

@@ -1 +1 @@
-../../.rspec
+test_apps/grape/../../.rspec

data/test_apps/grape/gemfiles/grape_0.16.2.gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: ../../..
   specs:
-    sapience (1.0.9)
+    sapience (1.0.10)
       concurrent-ruby (~> 1.0)
 
 GEM
@@ -137,4 +137,4 @@ DEPENDENCIES
   simplecov-json
 
 BUNDLED WITH
-   1.13.1
+   1.13.2

data/test_apps/grape/gemfiles/grape_0.17.0.gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: ../../..
   specs:
-    sapience (1.0.9)
+    sapience (1.0.10)
       concurrent-ruby (~> 1.0)
 
 GEM
@@ -135,4 +135,4 @@ DEPENDENCIES
   simplecov-json
 
 BUNDLED WITH
-   1.13.1
+   1.13.2

data/test_apps/grape/spec/lib/ping/api_spec.rb

@@ -12,7 +12,7 @@ describe Ping::API do
     let(:logger) { Grape::API.logger }
 
     specify do
-      get "/api/ping"
+      get "/api/ping", {}, "CONTENT-TYPE" => "application/json"
       expect(last_response.body).to match(/PONG/)
     end
 
@@ -34,7 +34,7 @@ describe Ping::API do
       expect(logger).to receive(:info).with(
         method:       "GET",
         request_path: "/api/ping",
-        format:       :json,
+        format:       "json",
         status:       200,
         class_name:   "Ping::API",
         action:       "index",
@@ -50,7 +50,7 @@ describe Ping::API do
                       ),
       )
 
-      get "/api/ping"
+      get "/api/ping", {}, "CONTENT-TYPE" => "application/json"
     end
 
     context "no routes defined" do
@@ -58,7 +58,7 @@ describe Ping::API do
         expect(logger).to receive(:info).with(
           method:       "GET",
           request_path: "/api/404",
-          format:       :json,
+          format:       "json",
           status:       404,
           class_name:   "Ping::API",
           action:       "index",
@@ -74,7 +74,7 @@ describe Ping::API do
                         ),
         )
 
-        get "/api/404"
+        get "/api/404", {}, "CONTENT-TYPE" => "application/json"
       end
     end
   end

data/test_apps/rails/.rspec

@@ -1 +1 @@
-../../.rspec
+test_apps/rails/../../.rspec

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: sapience
 version: !ruby/object:Gem::Version
-  version: 1.0.10
+  version: 1.0.11
 platform: ruby
 authors:
 - Mikael Henriksson
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-10-11 00:00:00.000000000 Z
+date: 2016-10-12 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: concurrent-ruby
@@ -456,7 +456,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.5.1
+rubygems_version: 2.6.7
 signing_key: 
 specification_version: 4
 summary: Hasslefree autoconfiguration for logging, metrics and exception collection.