logster 2.11.3 → 2.12.1

data/lib/logster/middleware/viewer.rb

@@ -18,7 +18,7 @@ module Logster
         (@store = Logster.store) || raise(ArgumentError.new("store"))
 
         @assets_path = File.expand_path("../../../../assets", __FILE__)
-        @fileserver = Rack::File.new(@assets_path)
+        @fileserver = Rack::Files.new(@assets_path)
       end
 
       def call(env)
@@ -33,7 +33,7 @@ module Logster
           if resource =~ /\.ico$|\.js$|\.png|\.handlebars$|\.css$|\.woff$|\.ttf$|\.woff2$|\.svg$|\.otf$|\.eot$/
             serve_file(env, resource)
 
-          elsif resource.start_with?("/messages.json")
+          elsif resource.start_with?("/messages.json") && env[REQUEST_METHOD] == "POST"
             serve_messages(Rack::Request.new(env))
 
           elsif resource =~ /\/message\/([0-9a-f]+)$/
@@ -102,10 +102,10 @@ module Logster
             end
 
             if json
-              [200, { "Content-Type" => "application/json; charset=utf-8" }, [message.to_json]]
+              [200, { "content-type" => "application/json; charset=utf-8" }, [message.to_json]]
             else
               preload = { "/show/#{key}" => message }
-              [200, { "Content-Type" => "text/html; charset=utf-8" }, [body(preload)]]
+              [200, { "content-type" => "text/html; charset=utf-8" }, [body(preload)]]
             end
 
           elsif resource =~ /\/settings(\.json)?$/
@@ -128,9 +128,9 @@ module Logster
               grouping = Logster::GroupingPattern.find_all(raw: true).map do |pattern|
                 { value: pattern }
               end
-              [200, { "Content-Type" => "application/json; charset=utf-8" }, [JSON.generate(suppression: suppression, grouping: grouping)]]
+              [200, { "content-type" => "application/json; charset=utf-8" }, [JSON.generate(suppression: suppression, grouping: grouping)]]
             else
-              [200, { "Content-Type" => "text/html; charset=utf-8" }, [body]]
+              [200, { "content-type" => "text/html; charset=utf-8" }, [body]]
             end
           elsif resource =~ /\/patterns\/([a-zA-Z0-9_]+)\.json$/
             unless Logster.config.enable_custom_patterns_via_ui
@@ -161,12 +161,12 @@ module Logster
             Logster.store.remove_ignore_count(pattern)
             [200, {}, ["OK"]]
           elsif resource == "/"
-            [200, { "Content-Type" => "text/html; charset=utf-8" }, [body]]
+            [200, { "content-type" => "text/html; charset=utf-8" }, [body]]
           elsif resource =~ /\/fetch-env\/([0-9a-f]+)\.json$/
             key = $1
             env = Logster.store.get_env(key)
             if env
-              [200, { "Content-Type" => "application/json; charset=utf-8" }, [JSON.generate(env)]]
+              [200, { "content-type" => "application/json; charset=utf-8" }, [JSON.generate(env)]]
             else
               not_found
             end
@@ -181,7 +181,7 @@ module Logster
             group.messages_keys.each { |k| Logster.store.solve(k) }
             [200, {}, []]
           elsif resource == '/development-preload.json' && ENV["LOGSTER_ENV"] == "development"
-            [200, { "Content-Type" => "application/json; charset=utf-8" }, [JSON.generate(preloaded_data)]]
+            [200, { "content-type" => "application/json; charset=utf-8" }, [JSON.generate(preloaded_data)]]
           else
             not_found
           end
@@ -230,7 +230,7 @@ module Logster
         }
 
         json = JSON.generate(payload)
-        [200, { "Content-Type" => "application/json" }, [json]]
+        [200, { "content-type" => "application/json" }, [json]]
       end
 
       def update_patterns(set_name, req)
@@ -258,7 +258,7 @@ module Logster
           return method_not_allowed(%w[POST PUT DELETE])
         end
 
-        [200, { "Content-Type" => "application/json" }, [JSON.generate(pattern: record.to_s)]]
+        [200, { "content-type" => "application/json" }, [JSON.generate(pattern: record.to_s)]]
       rescue => err
         error_message = err.message
 
@@ -293,7 +293,7 @@ module Logster
         if Array === allowed_methods
           allowed_methods = allowed_methods.join(", ")
         end
-        [405, { "Allow" => allowed_methods }, []]
+        [405, { "allow" => allowed_methods }, []]
       end
 
       def parse_regex(string)

data/lib/logster/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Logster
-  VERSION = "2.11.3"
+  VERSION = "2.12.1"
 end

data/logster.gemspec

@@ -17,9 +17,12 @@ Gem::Specification.new do |spec|
 
   spec.required_ruby_version = ">= 2.5.0"
 
-  spec.files = `git ls-files -z`.split("\x0").reject do |f|
-    f.start_with?("website") || f.start_with?("bin")
-  end
+  files =
+    `git ls-files -z`.split("\x0").reject { |f| f.start_with?(/website|bin/) }
+  files += Dir.glob("assets/javascript/*")
+  files += Dir.glob("assets/stylesheets/*")
+  spec.files = files
+
   spec.executables   = spec.files.grep(%r{^bin/}) { |f| File.basename(f) }
   spec.test_files    = spec.files.grep(%r{^(test|spec|features)/})
   spec.require_paths = ["lib"]

data/test/logster/middleware/test_viewer.rb

@@ -43,23 +43,29 @@ class TestViewer < Minitest::Test
   end
 
   def test_search_raceguard_s
-    response = request.get('/logsie/messages.json?search=searchkey')
+    response = request.post('/logsie/messages.json?search=searchkey')
     result = JSON.parse(response.body)
     assert_equal('searchkey', result['search'])
   end
 
   def test_search_raceguard_sr
-    response = request.get('/logsie/messages.json?search=/regex/&regex_search=true')
+    response = request.post('/logsie/messages.json?search=/regex/&regex_search=true')
     result = JSON.parse(response.body)
     assert_equal('/regex/', result['search'])
   end
 
   def test_search_raceguard_f
-    response = request.get("/logsie/messages.json?filter=0_1_2_3_4")
+    response = request.post("/logsie/messages.json?filter=0_1_2_3_4")
     result = JSON.parse(response.body)
     assert_equal([0, 1, 2, 3, 4], result['filter'])
   end
 
+  def test_search_does_not_respond_to_get_requests
+    response = request.get("/logsie/messages.json?filter=0_1_2_3_4")
+    assert_equal(404, response.status)
+    assert_equal("Not found", response.body)
+  end
+
   def test_regex_parse
     assert_equal(/hello/i, viewer.send(:parse_regex, '/hello/i'))
   end
@@ -304,7 +310,7 @@ class TestViewer < Minitest::Test
     ).each do |path|
       response = request.get(path)
       assert_equal(200, response.status)
-      assert_equal('application/javascript', response.headers['Content-Type'])
+      assert_equal('application/javascript', response.headers['content-type'])
     end
   end
 
@@ -315,7 +321,7 @@ class TestViewer < Minitest::Test
     ).each do |path|
       response = request.get(path)
       assert_equal(200, response.status)
-      assert_equal('text/css', response.headers['Content-Type'])
+      assert_equal('text/css', response.headers['content-type'])
     end
   end
 
@@ -335,7 +341,7 @@ class TestViewer < Minitest::Test
     Logster.store.clear_all
     env = { "b" => 1, "c" => 2 }
     msg = Logster.store.report(Logger::INFO, "test", "something hello", env: env)
-    response = request.get("/logsie/messages.json")
+    response = request.post("/logsie/messages.json")
     assert_equal(200, response.status)
     messages = JSON.parse(response.body)["messages"]
     assert_equal(1, messages.size)
@@ -348,7 +354,7 @@ class TestViewer < Minitest::Test
     Logster.store.clear_all
     env = { "b" => 1, "c" => 2 }
     msg = Logster.store.report(Logger::INFO, "test", "something hello", env: env)
-    response = request.get("/logsie/messages.json?search=something")
+    response = request.post("/logsie/messages.json?search=something")
     assert_equal(200, response.status)
     messages = JSON.parse(response.body)["messages"]
     assert_equal(1, messages.size)
@@ -387,7 +393,7 @@ class TestViewer < Minitest::Test
     %i[get head options].each do |m|
       response = request.public_send(m, "/logsie/solve-group", params: { regex: "/gotta be post/" })
       assert_equal(405, response.status)
-      assert_equal("POST", response.headers["Allow"])
+      assert_equal("POST", response.headers["allow"])
     end
     latest = Logster.store.latest
     assert_equal(1, latest.size)

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: logster
 version: !ruby/object:Gem::Version
-  version: 2.11.3
+  version: 2.12.1
 platform: ruby
 authors:
 - Sam Saffron
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2022-08-25 00:00:00.000000000 Z
+date: 2023-03-10 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -156,17 +156,25 @@ files:
 - assets/images/Icon-144_square.png
 - assets/images/icon_144x144.png
 - assets/images/icon_64x64.png
+- assets/javascript/.gitkeep
+- assets/javascript/chunk.143.2faa04830259ce9aa59a.js
+- assets/javascript/chunk.178.ca5ade1d8cbdbfbe6d72.js
+- assets/javascript/chunk.468.95dd450003497c781cb3.js
+- assets/javascript/chunk.916.85a3fc9d873df80f5ea5.js
 - assets/javascript/client-app.js
 - assets/javascript/vendor.js
+- assets/stylesheets/.gitkeep
 - assets/stylesheets/client-app.css
 - assets/stylesheets/vendor.css
 - build_client_app.sh
 - client-app/.editorconfig
 - client-app/.ember-cli
 - client-app/.eslintignore
-- client-app/.eslintrc.js
+- client-app/.eslintrc
 - client-app/.gitignore
-- client-app/.travis.yml
+- client-app/.prettierignore
+- client-app/.prettierrc
+- client-app/.template-lintrc.js
 - client-app/.watchmanconfig
 - client-app/README.md
 - client-app/app/app.js
@@ -181,7 +189,6 @@ files:
 - client-app/app/components/tab-contents.js
 - client-app/app/components/tabbed-section.js
 - client-app/app/components/time-formatter.js
-- client-app/app/components/update-time.js
 - client-app/app/controllers/index.js
 - client-app/app/controllers/show.js
 - client-app/app/helpers/logster-url.js
@@ -200,6 +207,7 @@ files:
 - client-app/app/routes/index.js
 - client-app/app/routes/settings.js
 - client-app/app/routes/show.js
+- client-app/app/services/events.js
 - client-app/app/styles/app.css
 - client-app/app/templates/application.hbs
 - client-app/app/templates/components/actions-menu.hbs
@@ -215,12 +223,12 @@ files:
 - client-app/app/templates/index.hbs
 - client-app/app/templates/settings.hbs
 - client-app/app/templates/show.hbs
+- client-app/config/ember-cli-update.json
 - client-app/config/environment.js
 - client-app/config/icons.js
 - client-app/config/optional-features.json
 - client-app/config/targets.js
 - client-app/ember-cli-build.js
-- client-app/package-lock.json
 - client-app/package.json
 - client-app/preload-json-manager.rb
 - client-app/public/assets/images/icon_144x144.png
@@ -236,6 +244,7 @@ files:
 - client-app/tests/unit/controllers/show-test.js
 - client-app/tests/unit/routes/index-test.js
 - client-app/tests/unit/routes/show-test.js
+- client-app/yarn.lock
 - lib/examples/sidekiq_logster_reporter.rb
 - lib/logster.rb
 - lib/logster/base_store.rb

data/client-app/.eslintrc.js

@@ -1,60 +0,0 @@
-module.exports = {
-  root: true,
-  parser: 'babel-eslint',
-  parserOptions: {
-    ecmaVersion: 2018,
-    sourceType: 'module',
-    ecmaFeatures: {
-      legacyDecorators: true
-    }
-  },
-  plugins: [
-    'ember'
-  ],
-  globals: {
-    Em: false,
-    Ember: false,
-    moment: false,
-    _: false
-  },
-  extends: [
-    'eslint:recommended',
-    'plugin:ember/recommended'
-  ],
-  env: {
-    browser: true
-  },
-  rules: {
-    'ember/no-jquery': 'error'
-  },
-  overrides: [
-    // node files
-    {
-      files: [
-        '.eslintrc.js',
-        '.template-lintrc.js',
-        'ember-cli-build.js',
-        'testem.js',
-        'blueprints/*/index.js',
-        'config/**/*.js',
-        'lib/*/index.js',
-        'server/**/*.js'
-      ],
-      parserOptions: {
-        sourceType: 'script'
-      },
-      env: {
-        browser: false,
-        node: true
-      },
-      plugins: ['node'],
-      rules: Object.assign({}, require('eslint-plugin-node').configs.recommended.rules, {
-        // add your custom rules and overrides for node files here
-
-        // this can be removed once the following is fixed
-        // https://github.com/mysticatea/eslint-plugin-node/issues/77
-        'node/no-unpublished-require': 'off'
-      })
-    }
-  ]
-};

data/client-app/.travis.yml

@@ -1,28 +0,0 @@
----
-language: node_js
-node_js:
-  - "8"
-
-sudo: false
-dist: trusty
-
-addons:
-  chrome: stable
-
-cache:
-  directories:
-    - $HOME/.npm
-
-env:
-  global:
-    # See https://git.io/vdao3 for details.
-    - JOBS=1
-
-branches:
-  only:
-    - master
-
-script:
-  - npm run lint:hbs
-  - npm run lint:js
-  - npm test

data/client-app/app/components/update-time.js

@@ -1,21 +0,0 @@
-import Component from "@ember/component";
-import { formatTime } from "client-app/lib/utilities";
-import { later } from "@ember/runloop";
-
-export default Component.extend({
-  didInsertElement() {
-    later(this, this.updateTimes, 60000);
-  },
-
-  updateTimes() {
-    Array.from(document.querySelectorAll(".auto-update-time")).forEach(node => {
-      const timestamp = parseInt(node.dataset.timestamp);
-      if (!timestamp) return;
-      const formatted = formatTime(timestamp);
-      if (formatted !== node.innerText) {
-        node.innerText = formatted;
-      }
-    });
-    later(this, this.updateTimes, 60000);
-  }
-});