pg_eventstore 1.0.0.rc1 → 1.0.0.rc2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: fbcfe058d0769d031f29580fdd79e899a73987a3108b0b96ad154a5bb913a7d2
-  data.tar.gz: 0f071621693bca4b597defc2be13fbde054be06448648ace7324e8ce14e4e512
+  metadata.gz: 84901d9b6a866b451fe90706312302ffd2bdbcd22a5a54cbff5c10123601336c
+  data.tar.gz: 2f1e79704c3823763a65b5ddf8554413c3c684782454b1e4e5eaf562ae9a1699
 SHA512:
-  metadata.gz: fa8828e002dab641f15b8ed3756c822899b3937929482a8fb9dcbac4bb5c10b7283427fa462a627ee47209d4476157396749ced15d50b64892683b1309e897a8
-  data.tar.gz: 3482263831bc256851999a0093be615cf4e238ceeae5c27f209f2fcc3d8b666db8b4c68bc388e5ef0c52de8f52a65e6abfe4db864042161f823b867f4996e750
+  metadata.gz: 2155de5400e32927278cdb79554b1f0276ebf3e134e8317f099d1b782bf626a3ad10c5dbdfbab0cd194abc59987a03d83dc7c0dae3a1449f35875a8fc17d6edc
+  data.tar.gz: e39889f475939611609d5f48c8742da829ea691b603affda17358ab27d2a4a0f5877c0dd186ce356ffc8951c3700fe59d2258b124205eca4b1bab972f755b868

data/CHANGELOG.md

@@ -1,5 +1,9 @@
 ## [Unreleased]
 
+## [1.0.0.rc2]
+
+- Implement confirmation dialog for sensitive admin UI actions
+
 ## [1.0.0.rc1]
 
 - Improve performance of loading original events when resolve_link_tos: true option is provided

data/lib/pg_eventstore/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module PgEventstore
-  VERSION = "1.0.0.rc1"
+  VERSION = "1.0.0.rc2"
 end

data/lib/pg_eventstore/web/application.rb

@@ -53,6 +53,13 @@ module PgEventstore
 
           "#{request.referer}#{params[:hash]}"
         end
+
+        # Shortcut to escape html
+        # @param text [String]
+        # @return [String]
+        def h(text)
+          Rack::Utils.escape_html(text)
+        end
       end
 
       get '/' do

data/lib/pg_eventstore/web/public/javascripts/pg_eventstore.js

@@ -136,19 +136,6 @@ $(function(){
         $(this).parents('tr').next().toggleClass('d-none');
     });
 
-    // Handle a[data-method]. It works very similar to rails ujs
-    $('body').on('click', 'a[data-method]', function(e){
-        e.preventDefault();
-
-        let href = $(this).attr('href');
-        let method = $(this).data('method');
-        let $form = $(`<form method="${method}" action="${href}"></form>`);
-        let hashInput = `<input name="hash" value="${window.location.hash}" type="hidden" />`;
-
-        $form.hide().append(hashInput).appendTo('body');
-        $form.submit();
-    });
-
     // When user navigates through SubscriptionsSet-s tabs - also change a hash of the url. Its value is send when
     // clicking on a[data-method] links
     $('.set-tab').click(function(e){
@@ -160,3 +147,51 @@ $(function(){
         $(`.set-tab[href="${window.location.hash}"]`).get(0).click();
     }
 });
+
+// Confirmation dialog and data-method handling functional
+$(function(){
+    "use strict";
+
+    let $confirmationModal = $('#confirmation-modal');
+
+    $confirmationModal.on('hide.bs.modal', function(){
+        $(this).find('.modal-title').html('');
+        $(this).find('.modal-body').html('');
+        $(this).find('.confirm').off();
+    });
+    let showConfirmation = function(el, callback){
+        let $el = $(el);
+        $confirmationModal.find('.modal-body').html($el.data('confirm'));
+        $confirmationModal.find('.modal-title').html($el.data('confirm-title'));
+        $confirmationModal.modal('show');
+        $confirmationModal.one('click', '.confirm', callback);
+    }
+
+    let handleMethod = function(el){
+        let $el = $(el);
+        let href = $el.attr('href');
+        let method = $el.data('method') || 'GET';
+
+        if(method === 'GET') {
+            window.location.href = href;
+        }else{
+            let $form = $(`<form method="${method}" action="${href}"></form>`);
+            let hashInput = `<input name="hash" value="${window.location.hash}" type="hidden" />`;
+
+            $form.append(hashInput);
+            $form.hide().appendTo('body');
+            $form.submit();
+        }
+    }
+
+    $('body').on('click', 'a[data-confirm], a[data-method]', function(e){
+        e.preventDefault();
+        if($(this).data('confirm')) {
+            showConfirmation(e.target, function(){
+                handleMethod(e.target);
+            });
+            return;
+        }
+        handleMethod(e.target);
+    });
+});

data/lib/pg_eventstore/web/public/stylesheets/pg_eventstore.css

@@ -3,3 +3,7 @@ tr.collapsing {
     transition: none;
     display: none;
 }
+
+#confirmation-modal .modal-body {
+    font-size: 1.2rem;
+}

data/lib/pg_eventstore/web/views/layouts/application.erb

@@ -101,6 +101,25 @@
   </div>
 </div>
 
+<div class="modal fade" id="confirmation-modal" tabindex="-1" role="dialog" aria-labelledby="confirmation-modal" aria-hidden="true">
+  <div class="modal-dialog modal-dialog-centered" role="document">
+    <div class="modal-content">
+      <div class="modal-header">
+        <h5 class="modal-title"></h5>
+        <button type="button" class="close" data-dismiss="modal" aria-label="Close">
+          <span aria-hidden="true">&times;</span>
+        </button>
+      </div>
+      <div class="modal-body font-weight-bold text-break">
+      </div>
+      <div class="modal-footer">
+        <button type="button" class="btn btn-secondary" data-dismiss="modal">Cancel</button>
+        <button type="button" class="btn btn-primary confirm">Continue</button>
+      </div>
+    </div>
+  </div>
+</div>
+
 <!-- jQuery -->
 <script src="javascripts/vendor/jquery.min.js"></script>
 <!-- Bootstrap -->

data/lib/pg_eventstore/web/views/subscriptions/index.erb

@@ -21,10 +21,6 @@
     <div class="row">
       <div class="col-md-12 col-sm-12">
         <div class="x_panel">
-          <div class="x_title">
-            <h2><i class="fa fa-bars"></i> Tabs <small>Float left</small></h2>
-            <div class="clearfix"></div>
-          </div>
           <div class="x_content">
             <ul class="nav nav-tabs bar_tabs" role="tablist">
               <% @association.association.each.with_index do |(subscriptions_set, _), index| %>
@@ -78,11 +74,11 @@
                                 </a>
                               <% end %>
                               <% if PgEventstore::RunnerState::STATES.values_at(:running, :dead).include?(subscriptions_set.state) %>
-                                <a class="btn btn-warning" data-method="post" href="<%= subscriptions_set_cmd_url(subscriptions_set.id, subscriptions_set_cmds[:Stop]) %>" data-toggle="tooltip" title="This action will delete Subscriptions Set and release all related Subscriptions.">
+                                <a class="btn btn-warning" data-confirm="You are about to stop SubscriptionsSet#<%= subscriptions_set.id %>. This will also delete it and will result in stopping all related Subscriptions. Continue?" data-confirm-title="Stop SubscriptionsSet" data-method="post" href="<%= subscriptions_set_cmd_url(subscriptions_set.id, subscriptions_set_cmds[:Stop]) %>" data-toggle="tooltip" title="This action will delete Subscriptions Set and release all related Subscriptions.">
                                   Stop
                                 </a>
                               <% end %>
-                              <a class="btn btn-danger" data-method="post" href="<%= url("/delete_subscriptions_set/#{subscriptions_set.id}") %>" data-toggle="tooltip" title="Use this action only on stuck Subscriptions Set - to clean it up.">
+                              <a class="btn btn-danger" data-confirm="You are about to delete SubscriptionsSet#<%= subscriptions_set.id %>. Continue?" data-confirm-title="Delete SubscriptionsSet" data-method="post" href="<%= url("/delete_subscriptions_set/#{subscriptions_set.id}") %>" data-toggle="tooltip" title="Use this action only on stuck Subscriptions Set - to clean it up.">
                                 Delete
                               </a>
                             </td>
@@ -108,11 +104,11 @@
                           <a class="btn btn-success" data-method="post" href="<%= subscriptions_set_cmd_url(subscriptions_set.id, subscriptions_set_cmds[:StartAll]) %>">
                             Start All
                           </a>
-                          <a class="btn btn-danger" data-method="post" href="<%= subscriptions_set_cmd_url(subscriptions_set.id, subscriptions_set_cmds[:StopAll]) %>">
+                          <a class="btn btn-danger" data-confirm="You are about to stop all Subscriptions. Continue?" data-confirm-title="Stop all Subscriptions" data-method="post" href="<%= subscriptions_set_cmd_url(subscriptions_set.id, subscriptions_set_cmds[:StopAll]) %>">
                             Stop All
                           </a>
                         <% else %>
-                          <a class="btn btn-danger" data-method="post" href="<%= delete_all_subscriptions_url(subscriptions.map(&:id)) %>" data-toggle="tooltip" title="Delete all Subscriptions, displayed on the current page.">
+                          <a class="btn btn-danger" data-confirm="You are about to delete all Subscriptions, listed on the page. Continue?" data-confirm-title="Delete all Subscriptions" data-method="post" href="<%= delete_all_subscriptions_url(subscriptions.map(&:id)) %>" data-toggle="tooltip" title="Delete all Subscriptions, displayed on the current page.">
                             Delete All
                           </a>
                         <% end %>
@@ -154,7 +150,11 @@
                             <td><%= subscription.chunk_query_interval %>s</td>
                             <td><%= subscription.last_chunk_fed_at %></td>
                             <td><%= colored_state(subscription.state, subscription.updated_at) %></td>
-                            <td><%= subscription.average_event_processing_time %></td>
+                            <td>
+                              <% if subscription.average_event_processing_time %>
+                                <%= "#{(1 / subscription.average_event_processing_time).to_i}/s" %>
+                              <% end %>
+                            </td>
                             <td><%= subscription.restart_count %></td>
                             <td><%= subscription.max_restarts_number %></td>
                             <td><%= subscription.time_between_restarts %>s</td>
@@ -186,7 +186,7 @@
                                 <% end %>
                               <% end %>
                               <% unless subscriptions_set.id %>
-                                <a class="btn btn-danger" data-method="post" href="<%= url("/delete_subscription/#{subscription.id}") %>" data-toggle="tooltip" title="You will lose the Subscription's position as well.">
+                                <a class="btn btn-danger" data-confirm="You are about to delete <%= h subscription.name.inspect %> Subscription. Continue?" data-confirm-title="Delete Subscription" data-method="post" href="<%= url("/delete_subscription/#{subscription.id}") %>" data-toggle="tooltip" title="You will lose the Subscription's position as well.">
                                   Delete
                                 </a>
                               <% end %>

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: pg_eventstore
 version: !ruby/object:Gem::Version
-  version: 1.0.0.rc1
+  version: 1.0.0.rc2
 platform: ruby
 authors:
 - Ivan Dzyzenko
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2024-04-05 00:00:00.000000000 Z
+date: 2024-04-10 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: pg