rack_mem_prof 0.0.2 → 0.0.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a18a36ef5f7dd70c156fd5b1fd75597cd9b3069032e5f5522c13a018197d9e32
-  data.tar.gz: 8c39106036de7812b16b9934fafc5678153b758d70a9418ff85e7804888f0c75
+  metadata.gz: 148ad3e85a052b4acf18e795977790a7862fa4dda7cdee2ff99b6557e12e736c
+  data.tar.gz: a8405090d40a1c2607088a240d6eeb615d4459e501d4f7d256ba82448efdf262
 SHA512:
-  metadata.gz: 64d47fee0f12c47c88ab71740e6708d2738da0612e6595e227ae572f4827c50477efd35cd718d602d65d180655f3b3eac404d51d4aa6010527aedfcb5545a8ef
-  data.tar.gz: a6d75f76fd72cd2689b2b57a62c1c0abb3797db71034d646b413f5bd2f530076524265bf3a6005c22f4f9523344065687a1f6847ed2d7d593dce8aa3b4f70954
+  metadata.gz: 6419abdeac526777a0900d00fde152dacd0f425bf3e3f1de2a0354b4cbd565e62a65183eded03bac8234e8f90123b7c1be1d368016f6f18fe6028b044a622480
+  data.tar.gz: 74762a5687acf6b840e7df7b93ee6bd2f98551594c2ad0942c47211120ba1af4d54e00300b98c2cc736d1b851288f854ee8cbb45f445579f2eab7ff1923b3fda

data/lib/rack/mem_prof/middleware.rb

@@ -1,13 +1,14 @@
+require 'digest'
 require 'fileutils'
 require 'memory_profiler'
 require 'rack'
-require 'pry'
 
 module Rack
   module MemProf
     class Middleware
       REPEXT = '.txt'.freeze
       REPDIR = 'rack_mem_prof'.freeze
+      UNSLUGGABLE = 'stub_for_unslaggable_requests'.freeze
 
       def initialize(app, options = {})
         @app = app
@@ -16,9 +17,20 @@ module Rack
       end
 
       def call(env)
-        MemoryProfiler.start
-        status, headers, body = @app.call(env)
-        write_report(MemoryProfiler.stop, env)
+        report_path, report_filename = build_report_path(env)
+        path_with_file = ::File.join(report_path, report_filename)
+
+        if ::File.exists?(path_with_file)
+          status, headers, body = @app.call(env)
+        else
+          FileUtils.mkdir_p report_path
+
+          MemoryProfiler.start
+
+          status, headers, body = @app.call(env)
+
+          write_report(MemoryProfiler.stop, path_with_file)
+        end
 
         [status, headers, body]
       rescue => e
@@ -29,27 +41,26 @@ module Rack
       private
 
       # report to "/tmp/rack_mem_prof/<request path>/<query params sha1>.txt"
-      def write_report(report, env)
-        request_report_path = ensure_path(env)
+      def write_report(report, path)
         report.pretty_print(
           scale_bytes: @scale_bytes,
-          to_file: report_file(request_report_path, env)
+          to_file: path,
         )
       end
 
-      def report_file(request_report_path, env)
-        filename = generate_filename(env['QUERY_STRING'])
-        ::File.join(request_report_path, filename)
+      def build_report_path(env)
+        return generate_path(env['PATH_INFO']), generate_filename(env['QUERY_STRING'])
       end
 
-      def ensure_path(env)
-        request_dirname = webpath_to_dirname(env['PATH_INFO'])
-        request_report_path = ::File.join(reports_path, request_dirname)
-        FileUtils.mkdir_p request_report_path
+      def generate_path(path_info)
+        dirname_for_request = generate_dirname(path_info)
+        ::File.join(reports_path, dirname_for_request)
       end
 
-      def webpath_to_dirname(path_info)
-        path_info.gsub(/[^0-9A-Za-z]/, '').downcase
+      def generate_dirname(path_info)
+        dirname = path_info.gsub(/[^0-9A-Za-z]/, '').downcase
+
+        return UNSLUGGABLE if dirname.length.zero?
       end
 
       def generate_filename(query_string)

data/lib/rack/mem_prof/version.rb

@@ -1,5 +1,5 @@
 module Rack
   module MemProf
-    VERSION = '0.0.2'
+    VERSION = '0.0.4'
   end
 end

data/rack_mem_prof.gemspec

@@ -27,7 +27,7 @@ Gem::Specification.new do |spec|
   spec.add_development_dependency 'test-unit', '~> 3.3'
   spec.add_development_dependency 'roda', '~> 3.29'
   spec.add_development_dependency 'mocha', '~> 1.11'
-  spec.add_development_dependency 'pry'
+  spec.add_development_dependency 'pry-byebug'
 end
 
 

data/test/rack_mem_prof_test.rb

@@ -1,3 +1,4 @@
+require 'fileutils'
 require 'rack/mem_prof'
 require 'test/unit'
 require 'rack/test'
@@ -16,10 +17,13 @@ class RackMemProfTest < Test::Unit::TestCase
 
   def test_report_created
     self.app = Rack::MemProf::Middleware.new(DummyApp.freeze.app)
+
     MemoryProfiler.expects(:start).once
     MemoryProfiler.expects(:stop).once
     app.expects(:write_report).once
+
     get '/'
+
     assert last_response.ok?
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: rack_mem_prof
 version: !ruby/object:Gem::Version
-  version: 0.0.2
+  version: 0.0.4
 platform: ruby
 authors:
 - Vitaly Shvedchenko
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-03-02 00:00:00.000000000 Z
+date: 2020-03-03 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rack
@@ -123,7 +123,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '1.11'
 - !ruby/object:Gem::Dependency
-  name: pry
+  name: pry-byebug
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="