sidekiq-status 2.0.0 → 2.1.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 8062f467160552b97566de033649c1b33869da92ce3ff168b1a20b1b38340eb6
-  data.tar.gz: 2a918b6cb05fe934db2521461b2abf26e4680def459486d4afe143e4a51d0cd6
+  metadata.gz: c02846f130e9fc47945e6e85e03d2e5cb5033e1d5a55f8b9ade921b72839f3a4
+  data.tar.gz: e236b4fd1fd3b94387b31786fe0aff021904b1947cf3bc2b26e66fceb6e38ca2
 SHA512:
-  metadata.gz: 2d35f5ab165082045af4e53220d72a161a821816a3ec8f8cd8cec7ce2811358b9d78b56d211fea0cae65b5990e7fbbb04f4264e2a3f09588ce9c642cca3a705b
-  data.tar.gz: 558584271417c9eb89112268fd779c75f47e90044dd94c799675bec0a0f325a6015000e56a3d64397ed21eee0ae2089b22ad03bd73ed059b1ada368592e42e81
+  metadata.gz: 274efd30dff5904a2c1b39c2940e6785f8a446aada35e2a6fe38e58ddbc22ce0866e84e487e2bebfa439ca7ed06f862ff30c15fd3df5c48a3b59f03a6eabb95b
+  data.tar.gz: 590bd3e108417ba70cfa7706885a524f014f80ead06aef364b113099620fb362cafa181c0640ef2adb9309891ed33b3bac41585944e9a091716bc4ae4cf553ef

data/.travis.yml

@@ -6,6 +6,7 @@ rvm:
   - ruby-head
 gemfile:
   - gemfiles/sidekiq_5.x.gemfile
+  - gemfiles/sidekiq_6.1.gemfile
   - gemfiles/sidekiq_6.x.gemfile
 before_install:
   - gem update --system

data/Appraisals

@@ -2,6 +2,10 @@ appraise "sidekiq-5.x" do
   gem "sidekiq", "~> 5"
 end
 
+appraise "sidekiq-6.1" do
+  gem "sidekiq", "~> 6.1"
+end
+
 appraise "sidekiq-6.x" do
   gem "sidekiq", "~> 6"
 end

data/CHANGELOG.md

@@ -1,8 +1,23 @@
+**Version 2.1.2**
+ * Casts values to strings when HTML-encoding
+
+**Version 2.1.1**
+ * Ensures parameter outputs are properly HTML-encoded
+
+**Version 2.1.0**
+ * Adds support for Sidekiq 6.2.2+ (https://github.com/mperham/sidekiq/issues/4955)
+
+**Version 2.0.2**
+ * Fixes for dark mode theme
+
+**Version 2.0.1**
+ * Adds support for dark mode to the job status page
+
 **Version 2.0.0**
-* Adds support for Ruby 2.7, 3.0
-* Adds support for Sidekiq 6.x
-* Removes support for Ruby 2.3, 2.4, 2.5
-* Removes support for Sidekiq 3.x, 4.x
+ * Adds support for Ruby 2.7, 3.0
+ * Adds support for Sidekiq 6.x
+ * Removes support for Ruby 2.3, 2.4, 2.5
+ * Removes support for Sidekiq 3.x, 4.x
 
 **Versions 1.1.4 and prior**
 

data/README.md

@@ -21,9 +21,9 @@ Or install it yourself as:
 gem install sidekiq-status
 ```
 
-### Migrating to Version 2.x from 1.x
+#### Migrating to Version 2.x from 1.x
 
-Version 2.0.0 was published in order to add support for Ruby 3.0 and Sidekiq 6.x and to remove support for versions of both that are now end-of-life. You should be able to upgrade cleanly from version 1.x to 2.x provided you are running Sidekiq 5.x or newer.
+Version 2.0.0 was published in order to add support for Ruby 3.0 and Sidekiq 6.x and to remove support for versions of both that are now end-of-life. **You should be able to upgrade cleanly from version 1.x to 2.x provided you are running Sidekiq 5.x or newer.**
 
 Sidekiq-status version 1.1.4 provides support all the way back to Sidekiq 3.x and was maintained at https://github.com/utgarda/sidekiq-status/.
 

data/gemfiles/sidekiq_6.1.gemfile

@@ -0,0 +1,7 @@
+# This file was generated by Appraisal
+
+source "https://rubygems.org"
+
+gem "sidekiq", "~> 6.1"
+
+gemspec path: "../"

data/lib/sidekiq-status/client_middleware.rb

@@ -1,4 +1,6 @@
 require 'sidekiq/api'
+JOB_CLASS = Sidekiq.constants.include?(:JobRecord) ? Sidekiq::JobRecord : Sidekiq::Job
+
 module Sidekiq::Status
 # Should be in the client middleware chain
   class ClientMiddleware
@@ -34,7 +36,7 @@ module Sidekiq::Status
         initial_metadata = {
           jid: msg['jid'],
           status: :queued,
-          worker: Sidekiq::Job.new(msg, queue).display_class,
+          worker: JOB_CLASS.new(msg, queue).display_class,
           args: display_args(msg, queue)
         }
         store_for_id msg['jid'], initial_metadata, job_class.new.expiration || @expiration, redis_pool
@@ -45,7 +47,7 @@ module Sidekiq::Status
     end
 
     def display_args(msg, queue)
-      job = Sidekiq::Job.new(msg, queue)
+      job = JOB_CLASS.new(msg, queue)
       return job.display_args.to_a.empty? ? nil : job.display_args.to_json
     rescue Exception => e
       # For Sidekiq ~> 2.7

data/lib/sidekiq-status/version.rb

@@ -1,5 +1,5 @@
 module Sidekiq
   module Status
-    VERSION = '2.0.0'
+    VERSION = '2.1.2'
   end
 end

data/sidekiq-status.gemspec

@@ -5,7 +5,7 @@ Gem::Specification.new do |gem|
   gem.authors       = ['Evgeniy Tsvigun', 'Kenaniah Cerny']
   gem.email         = ['utgarda@gmail.com', 'kenaniah@gmail.com']
   gem.summary       = 'An extension to the sidekiq message processing to track your jobs'
-  gem.homepage      = 'http://github.com/kenaniah/sidekiq-status'
+  gem.homepage      = 'https://github.com/kenaniah/sidekiq-status'
   gem.license       = 'MIT'
 
   gem.files         = `git ls-files`.split($\)

data/spec/lib/sidekiq-status/client_middleware_spec.rb

@@ -12,13 +12,13 @@ describe Sidekiq::Status::ClientMiddleware do
   describe "without :expiration parameter" do
 
     it "sets queued status" do
-      expect(StubJob.perform_async arg1: 'val1').to eq(job_id)
+      expect(StubJob.perform_async 'arg1' => 'val1').to eq(job_id)
       expect(redis.hget("sidekiq:status:#{job_id}", :status)).to eq('queued')
       expect(Sidekiq::Status::queued?(job_id)).to be_truthy
     end
 
     it "sets status hash ttl" do
-      expect(StubJob.perform_async arg1: 'val1').to eq(job_id)
+      expect(StubJob.perform_async 'arg1' => 'val1').to eq(job_id)
       expect(1..Sidekiq::Status::DEFAULT_EXPIRY).to cover redis.ttl("sidekiq:status:#{job_id}")
     end
 
@@ -50,7 +50,7 @@ describe Sidekiq::Status::ClientMiddleware do
     end
 
     it "overwrites default expiry value" do
-      StubJob.perform_async arg1: 'val1'
+      StubJob.perform_async 'arg1' => 'val1'
       expect((Sidekiq::Status::DEFAULT_EXPIRY+1)..huge_expiration).to cover redis.ttl("sidekiq:status:#{job_id}")
     end
 

data/spec/lib/sidekiq-status/server_middleware_spec.rb

@@ -10,7 +10,7 @@ describe Sidekiq::Status::ServerMiddleware do
       allow(SecureRandom).to receive(:hex).once.and_return(job_id)
       start_server do
         thread = redis_thread 4, "status_updates", "job_messages_#{job_id}"
-        expect(ConfirmationJob.perform_async arg1: 'val1').to eq(job_id)
+        expect(ConfirmationJob.perform_async 'arg1' => 'val1').to eq(job_id)
         expect(thread.value).to eq([
           job_id,
           job_id,
@@ -90,7 +90,7 @@ describe Sidekiq::Status::ServerMiddleware do
     it "sets status hash ttl" do
       allow(SecureRandom).to receive(:hex).once.and_return(job_id)
       start_server do
-        expect(StubJob.perform_async arg1: 'val1').to eq(job_id)
+        expect(StubJob.perform_async 'arg1' => 'val1').to eq(job_id)
       end
       expect(1..Sidekiq::Status::DEFAULT_EXPIRY).to cover redis.ttl("sidekiq:status:#{job_id}")
     end
@@ -104,7 +104,7 @@ describe Sidekiq::Status::ServerMiddleware do
 
     it "overwrites default expiry value" do
       start_server(:expiration => huge_expiration) do
-        StubJob.perform_async arg1: 'val1'
+        StubJob.perform_async 'arg1' => 'val1'
       end
       expect((Sidekiq::Status::DEFAULT_EXPIRY-1)..huge_expiration).to cover redis.ttl("sidekiq:status:#{job_id}")
     end
@@ -113,7 +113,7 @@ describe Sidekiq::Status::ServerMiddleware do
       overwritten_expiration = huge_expiration * 100
       allow_any_instance_of(StubJob).to receive(:expiration).and_return(overwritten_expiration)
       start_server(:expiration => huge_expiration) do
-        StubJob.perform_async arg1: 'val1'
+        StubJob.perform_async 'arg1' => 'val1'
       end
       expect((huge_expiration+1)..overwritten_expiration).to cover redis.ttl("sidekiq:status:#{job_id}")
     end

data/spec/lib/sidekiq-status/web_spec.rb

@@ -61,7 +61,7 @@ describe 'sidekiq status web' do
     get "/statuses/#{job_id}"
     expect(last_response).to be_ok
     expect(last_response.body).to match(/#{job_id}/)
-    expect(last_response.body).to match(/1,"another argument"/)
+    expect(last_response.body).to match(/1,"another argument"/)
     expect(last_response.body).to match(/working/)
   end
 

data/spec/lib/sidekiq-status_spec.rb

@@ -159,7 +159,7 @@ describe Sidekiq::Status do
     end
 
     it "retries failed jobs" do
-      allow(SecureRandom).to receive(:hex).and_return(retried_job_id)
+      allow(SecureRandom).to receive(:hex).exactly(3).times.and_return(retried_job_id)
       start_server do
         expect(capture_status_updates(3) {
           expect(RetriedJob.perform_async()).to eq(retried_job_id)

data/spec/support/test_jobs.rb

@@ -119,13 +119,13 @@ end
 
 class RetriedJob < StubJob
 
-  sidekiq_options 'retry' => true
+  sidekiq_options retry: true
   sidekiq_retry_in do |count| 3 end # 3 second delay > job timeout in test suite
 
-  def perform()
+  def perform
     Sidekiq.redis do |conn|
       key = "RetriedJob_#{jid}"
-      unless conn.exists key
+      if [0, false].include? conn.exists(key)
         conn.set key, 'tried'
         raise StandardError
       end

data/web/views/status.erb

@@ -1,3 +1,4 @@
+<% require 'cgi'; def h(v); CGI.escape_html(v.to_s); end %>
 <style>
   .progress {
     background-color: #C8E1ED;
@@ -14,9 +15,9 @@
 </style>
 
 <h3>
-  Job Status: <%= @status["jid"] %>
-  <span class='label label-<%= @status["label"] %>'>
-    <%= @status["status"] %>
+  Job Status: <%= h @status["jid"] %>
+  <span class='label label-<%= h @status["label"] %>'>
+    <%= h @status["status"] %>
   </span>
 </h3>
 
@@ -28,16 +29,16 @@
   </div>
 </div>
 
-<div class="panel panel-default">
+<div class="panel panel-default" style="background-color: inherit">
   <div class="panel-body">
-    <h4><%= @status["worker"] %></h4>
+    <h4><%= h @status["worker"] %></h4>
 
     <div class="row">
       <div class="col-sm-2">
         <strong>Arguments</strong>
       </div>
       <div class="col-sm-10">
-        <p><%= @status["args"].empty? ? "<i>none</i>" : @status["args"] %></p>
+        <p><%= @status["args"].empty? ? "<i>none</i>" : h(@status["args"]) %></p>
       </div>
     </div>
 
@@ -46,7 +47,7 @@
         <strong>Message</strong>
       </div>
       <div class="col-sm-10">
-        <p><%= @status["message"] || "<i>none</i>" %></p>
+        <p><%= h(@status["message"]) || "<i>none</i>" %></p>
       </div>
     </div>
 
@@ -75,9 +76,9 @@
           </div>
           <div class="col-sm-10">
             <% if val && val.include?("\n") %>
-              <pre><%= val %></pre>
+              <pre><%= h val %></pre>
             <% else  %>
-              <p><%= val || "<i>none</i>" %></p>
+              <p><%= h(val) || "<i>none</i>" %></p>
             <% end %>
           </div>
         </div>

data/web/views/status_not_found.erb

@@ -1,5 +1,6 @@
-<h3>Job Status: <%= params[:jid] %></h3>
+<% require 'cgi'; def h(v); CGI.escape_html(v.to_s); end %>
+<h3>Job Status: <%= h params[:jid] %></h3>
 
-<div class="alert alert-danger" role="alert">
+<div role="alert">
   <strong>Uh oh!</strong> That job can't be found. It may have expired already.
 </div>

data/web/views/statuses.erb

@@ -1,3 +1,4 @@
+<% require 'cgi'; def h(v); CGI.escape_html(v.to_s); end %>
 <style>
 .progress {
   background-color: #C8E1ED;
@@ -76,11 +77,11 @@ function setPerPage(select){
     </div>
   </div>
 </div>
-<table class="table table-hover table-bordered table-striped table-white">
+<table class="table table-hover table-bordered table-striped">
   <tr>
-    <% @headers.each do |h| %>
-      <th class="header <%= h[:class] %> header_<%= h[:id] %>">
-        <a href="<%= h[:url] %>"><%= h[:name] %></a>
+    <% @headers.each do |hdr| %>
+      <th class="header <%= h hdr[:class] %> header_<%= h hdr[:id] %>">
+        <a href="<%= h hdr[:url] %>"><%= h hdr[:name] %></a>
       </th>
     <% end %>
     <th class="header">
@@ -90,13 +91,13 @@ function setPerPage(select){
   <% @statuses.each do |container| %>
     <tr>
       <td>
-        <div title='<%= container["jid"] %>'><a href="<%= root_path %>statuses/<%= container["jid"] %>"><%= container["worker"] %></a></div>
+        <div title='<%= h container["jid"] %>'><a href="<%= root_path %>statuses/<%= h container["jid"] %>"><%= h container["worker"] %></a></div>
       </td>
       <td>
-        <div class='args' title='<%= container["jid"] %>'><%= container["args"] %></div>
+        <div class='args' title='<%= h container["jid"] %>'><%= h container["args"] %></div>
       </td>
       <td style='text-align: center;'>
-        <div class='label label-<%= container["label"] %>'><%= container["status"] %></div>
+        <div class='label label-<%= h container["label"] %>'><%= h container["status"] %></div>
       </td>
       <% secs = Time.now.to_i - container["update_time"].to_i %>
       <td style='text-align: center; white-space: nowrap;' title="<%= Time.at(container["update_time"].to_i) %>">
@@ -109,11 +110,11 @@ function setPerPage(select){
       <td>
         <div class="progress progress-striped" style="margin-bottom: 0">
           <div class='message' style='text-align:right; padding-right:0.5em; background-color: transparent; float:right;'>
-            <%= container["message"] %>
+            <%= h container["message"] %>
           </div>
           <% if container["pct_complete"].to_i > 0 %>
-            <div class="bar message" style="width: <%= container["pct_complete"] %>%;">
-              <%= container["pct_complete"] %>%
+            <div class="bar message" style="width: <%= h container["pct_complete"] %>%;">
+              <%= h container["pct_complete"] %>%
             </div>
           <% end %>
         </div>
@@ -121,7 +122,7 @@ function setPerPage(select){
       <td>
         <div class="actions">
           <form action="statuses" method="post">
-            <input type="hidden" name="jid" value="<%= container["jid"] %>" />
+            <input type="hidden" name="jid" value="<%= h container["jid"] %>" />
             <%= csrf_tag %>
             <% if container["status"] == "complete" %>
               <input type="hidden" name="_method" value="delete" />

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: sidekiq-status
 version: !ruby/object:Gem::Version
-  version: 2.0.0
+  version: 2.1.2
 platform: ruby
 authors:
 - Evgeniy Tsvigun
@@ -9,7 +9,7 @@ authors:
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-06-26 00:00:00.000000000 Z
+date: 2022-01-27 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: sidekiq
@@ -142,6 +142,7 @@ files:
 - README.md
 - Rakefile
 - gemfiles/sidekiq_5.x.gemfile
+- gemfiles/sidekiq_6.1.gemfile
 - gemfiles/sidekiq_6.x.gemfile
 - lib/sidekiq-status.rb
 - lib/sidekiq-status/client_middleware.rb
@@ -167,7 +168,7 @@ files:
 - web/views/status.erb
 - web/views/status_not_found.erb
 - web/views/statuses.erb
-homepage: http://github.com/kenaniah/sidekiq-status
+homepage: https://github.com/kenaniah/sidekiq-status
 licenses:
 - MIT
 metadata: {}
@@ -186,7 +187,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.2.15
+rubygems_version: 3.2.32
 signing_key:
 specification_version: 4
 summary: An extension to the sidekiq message processing to track your jobs