rorvswild 0.0.2 → 0.0.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 4e75768f7cf7390cfb3c36192245897160fb1529
-  data.tar.gz: 83570aeb82387bec304eb5998f49ac72c2c23f00
+  metadata.gz: f54f409b8f0d1dedb0891da5a680d7a701a67a2d
+  data.tar.gz: bb647ca3ce47cc27fd6279dd25fbabae7d48ab15
 SHA512:
-  metadata.gz: 73d40d8b2d14317b7fb16ee61942c6212580951c2d337fe407c5d9765e0076c7950006f2401f87985750fbbd6439dfcda40321bd394edf713f802e33c869fe52
-  data.tar.gz: 24ec5bad1f5b8f0dfcafcb37fc3b7b30c4581829673b540c294ff93d0cab79ec2f5f83694ad839dbceee28fb0ec5047b8268cc8ad248f4ffb8bbaa675c10dffb
+  metadata.gz: af4176a3d8b8e48db393dfb0001c616184a98ca885163fb566f98bd089deac147eea471a372d0821d4120153088d0ec83af23007d69aad557e2f9d1d6b9ea185
+  data.tar.gz: 3b35e6030188581d2a8e239aa7163742665fd6401f38788f8046e62668fd79ba13858466254a22a2a32f631519f03562b721dcd92c52bb24daaf839f7b03066f

data/lib/rorvswild/version.rb

@@ -1,3 +1,3 @@
 class Rorvswild
-  VERSION = "0.0.2"
+  VERSION = "0.0.3"
 end

data/lib/rorvswild.rb

@@ -45,20 +45,24 @@ module RorVsWild
       request[:other_runtime] = compute_duration(start, finish) - request[:db_runtime] - request[:view_runtime]
       request[:params] = params_filter.filter(payload[:params]) if error
       Thread.new { post_request }
-    rescue => ex
-      Rails.logger.error("[RorVsWild] " + ex.inspect)
-      Rails.logger.error("[RorVsWild] " + ex.backtrace.join("\n[RorVsWild] "))
+    rescue => exception
+      log_error(exception)
     end
 
     def after_sql_query(name, start, finish, id, payload)
       return if !queries || payload[:name] == "EXPLAIN".freeze
       runtime, sql, plan = compute_duration(start, finish), nil, nil
       file, line, method = extract_file_and_line_from_call_stack(caller)
+
+      if !file # I have no idea of the origin of the query, so at least we log the SQL.
+        file, line, method = ["Unknow", 0, "Unknow"]
+        sql = payload[:sql]
+      end
+
       plan = explain(sql = payload[:sql], payload[:binds]) if runtime > sql_threshold
       queries << {file: file, line: line, sql: sql, plan: plan, runtime: runtime}
-    rescue => ex
-      Rails.logger.error("[RorVsWild] " + ex.inspect)
-      Rails.logger.error("[RorVsWild] " + ex.backtrace.join("\n[RorVsWild] "))
+    rescue => exception
+      log_error(exception)
     end
 
     def after_view_rendering(name, start, finish, id, payload)
@@ -107,11 +111,15 @@ module RorVsWild
 
     def post_request
       post("/requests", request: request.merge(queries: slowest_queries, views: slowest_views, error: error))
+    rescue => exception
+      log_error(exception)
     end
 
     def extract_file_and_line_from_call_stack(stack)
-      file, line, method = (stack.find { |str| str.include?(Rails.root.to_s) } ||
-        stack.find { |str| str.include?("(irb):") }).split(":")
+      location = stack.find { |str| str.include?(Rails.root.to_s) } ||
+        stack.find { |str| !str.include?("/active_support/notifications/") }  # Origin of the call comes from a gem probably
+      return unless location # Ok I have no idea
+      file, line, method = location.split(":")
       file.sub!(Rails.root.to_s, "")
       method.sub!("block in ", "")
       method.sub!("in `", "")
@@ -130,7 +138,7 @@ module RorVsWild
     def post(path, data)
       uri = URI(api_url + path)
       Net::HTTP.start(uri.host, uri.port) do |http|
-        post = Net::HTTP::Post.new(uri)
+        post = Net::HTTP::Post.new(uri.path)
         post.content_type = "application/json"
         post.basic_auth(app_id, api_key)
         post.body = data.to_json
@@ -141,5 +149,14 @@ module RorVsWild
     def params_filter
       @params_filter ||= ActionDispatch::Http::ParameterFilter.new(Rails.application.config.filter_parameters)
     end
+
+    def logger
+      Rails.logger
+    end
+
+    def log_error(exception)
+      logger.error("[RorVsWild] " + exception.inspect)
+      logger.error("[RorVsWild] " + exception.backtrace.join("\n[RorVsWild] "))
+    end
   end
 end

data/rorvswild.gemspec

@@ -8,8 +8,8 @@ Gem::Specification.new do |spec|
   spec.version       = Rorvswild::VERSION
   spec.authors       = ["Alexis Bernard"]
   spec.email         = ["alexis@bernard.io"]
-  spec.summary       = "RorVsWild tracks response time of your Rails application."
-  spec.description   = "RorVsWild helps you to improve the response time, by pointing out any slow request, query and view."
+  spec.summary       = "RorVsWild tracks response time of Rails applications."
+  spec.description   = "RorVsWild helps to improve the response time, by pointing out any slow request, query and view."
   spec.homepage      = "http://www.rorvswild.com"
   spec.license       = "MIT"
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: rorvswild
 version: !ruby/object:Gem::Version
-  version: 0.0.2
+  version: 0.0.3
 platform: ruby
 authors:
 - Alexis Bernard
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-10-06 00:00:00.000000000 Z
+date: 2014-10-24 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -24,8 +24,8 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '1.6'
-description: RorVsWild helps you to improve the response time, by pointing out any
-  slow request, query and view.
+description: RorVsWild helps to improve the response time, by pointing out any slow
+  request, query and view.
 email:
 - alexis@bernard.io
 executables: []
@@ -63,5 +63,5 @@ rubyforge_project:
 rubygems_version: 2.2.2
 signing_key: 
 specification_version: 4
-summary: RorVsWild tracks response time of your Rails application.
+summary: RorVsWild tracks response time of Rails applications.
 test_files: []