sidekiq-status 2.1.0 → 2.1.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: fc3436d5bc785da39c9f4f9d56d8179d75acc534935b9cab980ef7266bcdbe94
-  data.tar.gz: 373f55bc1dc6cb235fe1f858ceb6ccb49fbe4e8864e9dd1951c83cd1ae053e69
+  metadata.gz: c02846f130e9fc47945e6e85e03d2e5cb5033e1d5a55f8b9ade921b72839f3a4
+  data.tar.gz: e236b4fd1fd3b94387b31786fe0aff021904b1947cf3bc2b26e66fceb6e38ca2
 SHA512:
-  metadata.gz: 30ac2390e17ceb73ace6481921e28ad42ab2111012007ff2577a997ca7f34a6e8da59f06fa856f7abcbb07086c1863110417a6c2063399af672ee3e17fc227ad
-  data.tar.gz: b0084cbfdd76b4215d6787b5d910d84722bc538de84ef0f0663990a2dd0ec2ed3bc3e8aee9b71b50336517c11c86843c5abd190964c6321e5a0c27f8353f3fb3
+  metadata.gz: 274efd30dff5904a2c1b39c2940e6785f8a446aada35e2a6fe38e58ddbc22ce0866e84e487e2bebfa439ca7ed06f862ff30c15fd3df5c48a3b59f03a6eabb95b
+  data.tar.gz: 590bd3e108417ba70cfa7706885a524f014f80ead06aef364b113099620fb362cafa181c0640ef2adb9309891ed33b3bac41585944e9a091716bc4ae4cf553ef

data/CHANGELOG.md

@@ -1,3 +1,12 @@
+**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
 

data/lib/sidekiq-status/version.rb

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

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/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>
 
@@ -30,14 +31,14 @@
 
 <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,4 +1,5 @@
-<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 role="alert">
   <strong>Uh oh!</strong> That job can't be found. It may have expired already.

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;
@@ -78,9 +79,9 @@ function setPerPage(select){
 </div>
 <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.1.0
+  version: 2.1.2
 platform: ruby
 authors:
 - Evgeniy Tsvigun
@@ -9,7 +9,7 @@ authors:
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-09-23 00:00:00.000000000 Z
+date: 2022-01-27 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: sidekiq
@@ -187,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