blazer_xlsx 3.0.6 → 3.0.8

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 4309fcc7bc80c9b7d6ff7bdc8fa3dc93533b178835c05025e3969cf2a5dd3526
-  data.tar.gz: 723244fd829079c8a26a215343af2da7a3ff712cc7d0982d303f907591ae6158
+  metadata.gz: 5028ca491b2786dd3d421258bb044d35c4d8ea3bafa76a13412ce97c97b7e345
+  data.tar.gz: ebb2f929232d742569000d3ee4973a6fd955b0b5c41e60a8b1927e743fa57f69
 SHA512:
-  metadata.gz: 88dd7fb66ff8cae7ad3a158506d7e7fe6140c9a763016974d7feb6c49b63138f85c3054f16069417266c36b9c205c918fd5c78265375095fade173faa82ba0dd
-  data.tar.gz: 663d107a09f63ac46902d373591b3d5679bd4ebf71bb83a481870ef73f7fbdc6b55f8509993850cea1b4e7f5a3c86fd5b3dce4575a55172053bf7b5295e0ae5d
+  metadata.gz: 946dd43ae310c7243ae38acc8d72e0eb641e91b5a4612d9891db135222386426c82288c64874a2e7c40d1afa75cb67dae72c15dda2bb0ae3957f8c7a0850840b
+  data.tar.gz: a4f595cbd4497e43fcd696a65a231fc29e5a8ad32f81157864999f3e4a947e930d333d4f0f21e96d8d036fd2d3325e2cf2df29c98342331026a137eec31f23c8

data/CHANGELOG.md

@@ -1,6 +1,6 @@
 ## 3.0.4 (unreleased)
 
-- Fixed error with MySQL and binary data
+- Improved CSP support
 
 ## 3.0.3 (2024-01-10)
 

data/README.md

@@ -1026,51 +1026,6 @@ override_csp: true
 
 Maps now use Mapbox GL JS v1 instead of Mapbox.js, which affects Mapbox billing.
 
-### 2.6
-
-Custom adapters now need to specify how to quote variables in queries (there is no longer a default)
-
-```ruby
-class FooAdapter < Blazer::Adapters::BaseAdapter
-  def quoting
-    :backslash_escape # single quote strings and convert ' to \' and \ to \\
-    # or
-    :single_quote_escape # single quote strings and convert ' to ''
-    # or
-    ->(value) { ... } # custom method
-  end
-end
-```
-
-### 2.3
-
-To archive queries, create a migration
-
-```sh
-rails g migration add_status_to_blazer_queries
-```
-
-with:
-
-```ruby
-add_column :blazer_queries, :status, :string
-Blazer::Query.update_all(status: "active")
-```
-
-### 2.0
-
-To use Slack notifications, create a migration
-
-```sh
-rails g migration add_slack_channels_to_blazer_checks
-```
-
-with:
-
-```ruby
-add_column :blazer_checks, :slack_channels, :text
-```
-
 ## History
 
 View the [changelog](https://github.com/ankane/blazer/blob/master/CHANGELOG.md)

data/app/controllers/blazer/base_controller.rb

@@ -71,9 +71,9 @@ module Blazer
       if smart_var_data_source
         query = smart_var_data_source.smart_variables[var]
 
-        if query.is_a? Hash
-          smart_var = query.map { |k,v| [v, k] }
-        elsif query.is_a? Array
+        if query.is_a?(Hash)
+          smart_var = query.map { |k, v| [v, k] }
+        elsif query.is_a?(Array)
           smart_var = query.map { |v| [v, v] }
         elsif query
           result = smart_var_data_source.run_statement(query)

data/app/views/blazer/_variables.html.erb

@@ -1,6 +1,6 @@
 <% if @bind_vars.any? %>
   <% var_params = request.query_parameters %>
-  <script>
+  <%= javascript_tag nonce: true do %>
     <%= blazer_js_var "timeZone", Blazer.time_zone.tzinfo.name %>
     var now = moment.tz(timeZone)
     var format = "YYYY-MM-DD"
@@ -8,7 +8,7 @@
     function toDate(time) {
       return moment.tz(time.format(format), timeZone)
     }
-  </script>
+  <% end %>
   <form id="bind" method="get" action="<%= action %>" class="form-inline" style="margin-bottom: 15px;">
     <% date_vars = ["start_time", "end_time"] %>
     <% if (date_vars - @bind_vars).empty? %>
@@ -21,11 +21,11 @@
       <%= label_tag var, var %>
       <% if (data = @smart_vars[var]) %>
         <%= select_tag var, options_for_select([[nil, nil]] + data, selected: var_params[var]), style: "margin-right: 20px; width: 200px; display: none;" %>
-        <script>
+        <%= javascript_tag nonce: true do %>
           $("#<%= var %>").selectize({
             create: true
           });
-        </script>
+        <% end %>
       <% elsif var.end_with?("_at") || var == "start_time" || var == "end_time" %>
         <%= hidden_field_tag var, var_params[var] %>
 
@@ -35,7 +35,7 @@
           </div>
         </div>
 
-        <script>
+        <%= javascript_tag nonce: true do %>
           (function() {
             var input = $("#<%= var %>")
             var datePicker = $("#<%= var %>-select")
@@ -57,7 +57,7 @@
               datePicker.find("span").html(toDate(picker.startDate).format("MMMM D, YYYY"))
             }
           })()
-        </script>
+        <% end %>
       <% else %>
         <%= text_field_tag var, var_params[var], style: "width: 120px; margin-right: 20px;", autofocus: i == 0 && !var.end_with?("_at") && !var_params[var], class: "form-control" %>
       <% end %>
@@ -75,7 +75,7 @@
         </div>
       </div>
 
-      <script>
+      <%= javascript_tag nonce: true do %>
         function dateStr(daysAgo) {
           return now.clone().subtract(daysAgo || 0, "days").format(format)
         }
@@ -119,7 +119,7 @@
           $("#reportrange").trigger("apply.daterangepicker", picker)
           submitIfCompleted($("#start_time").closest("form"))
         }
-      </script>
+      <% end %>
     <% end %>
 
     <input type="submit" class="btn btn-success" value="Run" style="vertical-align: top;" />

data/app/views/blazer/checks/_form.html.erb

@@ -12,12 +12,12 @@
     <div class="hide">
       <%= f.select :query_id, [], {include_blank: true} %>
     </div>
-    <script>
+    <%= javascript_tag nonce: true do %>
       <%= blazer_js_var "queries", Blazer::Query.active.named.order(:name).select("id, name").map { |q| {text: q.name, value: q.id} } %>
       <%= blazer_js_var "items", [@check.query_id].compact %>
 
       $("#check_query_id").selectize({options: queries, items: items, highlight: false, maxOptions: 100}).parents(".hide").removeClass("hide");
-    </script>
+    <% end %>
   </div>
 
   <% if @check.respond_to?(:check_type) %>
@@ -28,9 +28,9 @@
         <% check_options << ["Anomaly (most recent data point)", "anomaly"] if Blazer.anomaly_checks %>
         <%= f.select :check_type, check_options %>
       </div>
-      <script>
+      <%= javascript_tag nonce: true do %>
         $("#check_check_type").selectize({}).parent().removeClass("hide");
-      </script>
+      <% end %>
     </div>
   <% elsif @check.respond_to?(:invert) %>
     <div class="form-group">
@@ -38,9 +38,9 @@
       <div class="hide">
         <%= f.select :invert, [["Any results (bad data)", false], ["No results (missing data)", true]] %>
       </div>
-      <script>
+      <%= javascript_tag nonce: true do %>
         $("#check_invert").selectize({}).parent().removeClass("hide");
-      </script>
+      <% end %>
     </div>
   <% end %>
 
@@ -50,9 +50,9 @@
       <div class="hide">
         <%= f.select :schedule, Blazer.check_schedules.map { |v| [v, v] } %>
       </div>
-      <script>
+      <%= javascript_tag nonce: true do %>
         $("#check_schedule").selectize({}).parent().removeClass("hide");
-      </script>
+      <% end %>
     </div>
   <% end %>
 

data/app/views/blazer/checks/index.html.erb

@@ -62,11 +62,11 @@
   </tbody>
 </table>
 
-<script>
+<%= javascript_tag nonce: true do %>
   $("#search").on("keyup", function() {
     var value = $(this).val().toLowerCase()
     $("#checks tbody tr").filter( function() {
       $(this).toggle($(this).text().toLowerCase().indexOf(value) > -1)
     })
   }).focus()
-</script>
+<% end %>

data/app/views/blazer/dashboards/_form.html.erb

@@ -30,7 +30,7 @@
   </p>
 <% end %>
 
-<script>
+<%= javascript_tag nonce: true do %>
   <%= blazer_js_var "queries", Blazer::Query.active.named.order(:name).select("id, name").map { |q| {text: q.name, value: q.id} } %>
   <%= blazer_js_var "dashboardQueries", @queries || @dashboard.dashboard_queries.order(:position).map(&:query) %>
 
@@ -79,4 +79,4 @@
       app.queries.splice(e.newIndex, 0, app.queries.splice(e.oldIndex, 1)[0])
     }
   })
-</script>
+<% end %>

data/app/views/blazer/dashboards/show.html.erb

@@ -38,7 +38,7 @@
       <p class="text-muted">Loading...</p>
     </div>
   </div>
-  <script>
+  <%= javascript_tag nonce: true do %>
     <% data = {statement: query.statement, query_id: query.id, data_source: query.data_source, variables: variable_params(query), only_chart: true} %>
     <% data.merge!(cohort_period: params[:cohort_period]) if params[:cohort_period] %>
     <%= blazer_js_var "data", data %>
@@ -49,5 +49,5 @@
     }, function (message) {
       $("#chart-<%= i %>").addClass("query-error").html(message)
     });
-  </script>
+  <% end %>
 <% end %>

data/app/views/blazer/queries/_form.html.erb

@@ -68,7 +68,7 @@
   </div>
 </div>
 
-<script>
+<%= javascript_tag nonce: true do %>
   <%= blazer_js_var "variableParams", @variable_params %>
   <%= blazer_js_var "previewStatement", Blazer.data_sources.to_h { |k, v| [k, (v.preview_statement rescue "")] } %>
 
@@ -252,4 +252,4 @@
   })
   app.config.compilerOptions.whitespace = "preserve"
   app.mount("#app")
-</script>
+<% end %>

data/app/views/blazer/queries/home.html.erb

@@ -56,7 +56,7 @@
   <p v-if="more" class="text-muted">Loading...</p>
 </div>
 
-<script>
+<%= javascript_tag nonce: true do %>
   <%= blazer_js_var "dashboards", @dashboards %>
   <%= blazer_js_var "queries", @queries %>
   <%= blazer_js_var "more", @more %>
@@ -166,4 +166,4 @@
   })
   app.config.compilerOptions.whitespace = "preserve"
   app.mount("#queries")
-</script>
+<% end %>

data/app/views/blazer/queries/run.html.erb

@@ -75,21 +75,21 @@
     <% if @markers.any? %>
       <% map_id = SecureRandom.hex %>
       <%= content_tag :div, nil, id: map_id, style: "height: #{@only_chart ? 300 : 500}px;" %>
-      <script>
+      <%= javascript_tag nonce: true do %>
         <%= blazer_js_var "mapboxAccessToken", Blazer.mapbox_access_token %>
         <%= blazer_js_var "markers", @markers %>
         <%= blazer_js_var "mapId", map_id %>
         new Mapkick.Map(mapId, markers, {accessToken: mapboxAccessToken, tooltips: {hover: false, html: true}});
-      </script>
+      <% end %>
     <% elsif @geojson.any? %>
       <% map_id = SecureRandom.hex %>
       <%= content_tag :div, nil, id: map_id, style: "height: #{@only_chart ? 300 : 500}px;" %>
-      <script>
+      <%= javascript_tag nonce: true do %>
         <%= blazer_js_var "mapboxAccessToken", Blazer.mapbox_access_token %>
         <%= blazer_js_var "geojson", @geojson %>
         <%= blazer_js_var "mapId", map_id %>
         new Mapkick.AreaMap(mapId, geojson, {accessToken: mapboxAccessToken, tooltips: {hover: false, html: true}});
-      </script>
+      <% end %>
     <% elsif chart_type == "line" %>
       <% chart_data = @columns[1..-1].each_with_index.map{ |k, i| {name: blazer_series_name(k), data: @rows.map{ |r| [r[0], r[i + 1]] }, library: series_library[i]} } %>
       <%= line_chart chart_data, **chart_options %>

data/app/views/blazer/queries/schema.html.erb

@@ -28,7 +28,7 @@
   </table>
 <% end %>
 
-<script>
+<%= javascript_tag nonce: true do %>
   $("#search").on("keyup", function() {
     var value = $(this).val().toLowerCase()
     $(".schema-table").filter(function() {
@@ -52,4 +52,4 @@
       $(this).toggle(found)
     })
   }).focus()
-</script>
+<% end %>

data/app/views/blazer/queries/show.html.erb

@@ -46,7 +46,7 @@
     <p class="text-muted">Loading...</p>
   </div>
 
-  <script>
+  <%= javascript_tag nonce: true do %>
     function showRun(data) {
       $("#results").html(data)
       $("#results table").stupidtable(stupidtableCustomSettings).stickyTableHeaders({fixedOffset: 60})
@@ -59,14 +59,14 @@
     <%= blazer_js_var "data", @run_data %>
 
     runQuery(data, showRun, showError)
-  </script>
+  <% end %>
 <% end %>
 
-<script>
+<%= javascript_tag nonce: true do %>
   // do not highlight really long queries
   // this can lead to performance issues
   var code = $("#code code")
   if (code.text().length < 10000) {
     hljs.highlightElement(code.get(0))
   }
-</script>
+<% end %>

data/app/views/blazer/uploads/index.html.erb

@@ -45,11 +45,11 @@
   </tbody>
 </table>
 
-<script>
+<%= javascript_tag nonce: true do %>
   $("#search").on("keyup", function() {
     var value = $(this).val().toLowerCase()
     $("#uploads tbody tr").filter( function() {
       $(this).toggle($(this).text().toLowerCase().indexOf(value) > -1)
     })
   }).focus()
-</script>
+<% end %>

data/app/views/layouts/blazer/application.html.erb

@@ -7,18 +7,18 @@
     <%= favicon_link_tag "blazer/favicon.png" %>
     <% if defined?(Propshaft::Railtie) %>
       <%= stylesheet_link_tag "blazer/bootstrap-propshaft", "blazer/bootstrap", "blazer/selectize", "blazer/github", "blazer/daterangepicker", "blazer/application" %>
-      <%= javascript_include_tag "blazer/jquery", "blazer/rails-ujs", "blazer/stupidtable", "blazer/stupidtable-custom-settings", "blazer/jquery.stickytableheaders", "blazer/selectize", "blazer/highlight.min", "blazer/moment", "blazer/moment-timezone-with-data", "blazer/daterangepicker", "blazer/chart.umd", "blazer/chartjs-adapter-date-fns.bundle", "blazer/chartkick", "blazer/mapkick.bundle", "blazer/ace/ace", "blazer/ace/ext-language_tools", "blazer/ace/theme-twilight", "blazer/ace/mode-sql", "blazer/ace/snippets/text", "blazer/ace/snippets/sql", "blazer/Sortable", "blazer/bootstrap", "blazer/vue.global.prod", "blazer/routes", "blazer/queries", "blazer/fuzzysearch", "blazer/application" %>
+      <%= javascript_include_tag "blazer/jquery", "blazer/rails-ujs", "blazer/stupidtable", "blazer/stupidtable-custom-settings", "blazer/jquery.stickytableheaders", "blazer/selectize", "blazer/highlight.min", "blazer/moment", "blazer/moment-timezone-with-data", "blazer/daterangepicker", "blazer/chart.umd", "blazer/chartjs-adapter-date-fns.bundle", "blazer/chartkick", "blazer/mapkick.bundle", "blazer/ace/ace", "blazer/ace/ext-language_tools", "blazer/ace/theme-twilight", "blazer/ace/mode-sql", "blazer/ace/snippets/text", "blazer/ace/snippets/sql", "blazer/Sortable", "blazer/bootstrap", "blazer/vue.global.prod", "blazer/routes", "blazer/queries", "blazer/fuzzysearch", "blazer/application", nonce: true %>
     <% else %>
       <%= stylesheet_link_tag "blazer/application" %>
-      <%= javascript_include_tag "blazer/application" %>
+      <%= javascript_include_tag "blazer/application", nonce: true %>
     <% end %>
-    <script>
+    <%= javascript_tag nonce: true do %>
       <%= blazer_js_var "rootPath", root_path %>
-    </script>
+    <% end %>
     <%= csrf_meta_tags %>
   </head>
   <body>
-    <div class="container">
+    <div class="container" style="margin-left:40px">
       <%= yield %>
     </div>
   </body>

data/lib/blazer/adapters/athena_adapter.rb

@@ -33,7 +33,7 @@ module Blazer
             # use token so we fetch cached results after query is run
             client_request_token: request_token,
             query_execution_context: {
-              database: database,
+              database: database
             }
           }
 

data/lib/blazer/adapters/hive_adapter.rb

@@ -45,7 +45,7 @@ module Blazer
             port: uri.port,
             username: uri.user,
             password: uri.password,
-            database: uri.path.sub(/\A\//, ""),
+            database: uri.path.delete_prefix("/"),
             mode: uri.scheme.to_sym
           )
         end

data/lib/blazer/adapters/presto_adapter.rb

@@ -39,10 +39,10 @@ module Blazer
       def client
         @client ||= begin
           uri = URI.parse(settings["url"])
-          query = uri.query ? CGI::parse(uri.query) : {}
+          query = uri.query ? CGI.parse(uri.query) : {}
           Presto::Client.new(
             server: "#{uri.host}:#{uri.port}",
-            catalog: uri.path.to_s.sub(/\A\//, ""),
+            catalog: uri.path.to_s.delete_prefix("/"),
             schema: query["schema"] || "public",
             user: uri.user,
             http_debug: false

data/lib/blazer/adapters/sql_adapter.rb

@@ -46,20 +46,6 @@ module Blazer
           if adapter_name == "Trilogy"
             columns.map! { |k| k.dup.force_encoding(Encoding::UTF_8) }
           end
-
-          # fix for binary data
-          if mysql?
-            rows =
-              rows.map do |row|
-                row.map do |v|
-                  if v.is_a?(String) && v.encoding == Encoding::BINARY
-                    "0x#{v.unpack1("H*").upcase}"
-                  else
-                    v
-                  end
-                end
-              end
-          end
         rescue => e
           error = e.message.sub(/.+ERROR: /, "")
           error = Blazer::TIMEOUT_MESSAGE if Blazer::TIMEOUT_ERRORS.any? { |e| error.include?(e) }
@@ -164,7 +150,7 @@ module Blazer
       def cohort_analysis_statement(statement, period:, days:)
         raise "Cohort analysis not supported" unless supports_cohort_analysis?
 
-        cohort_column = statement =~ /\bcohort_time\b/ ? "cohort_time" : "conversion_time"
+        cohort_column = statement.match?(/\bcohort_time\b/) ? "cohort_time" : "conversion_time"
         tzname = Blazer.time_zone.tzinfo.name
 
         if mysql?

data/lib/blazer/data_source.rb

@@ -107,7 +107,7 @@ module Blazer
       end
 
       unless result
-        comment = "blazer"
+        comment = "blazer".dup
         if options[:user].respond_to?(:id)
           comment << ",user_id:#{options[:user].id}"
         end

data/lib/blazer/result.rb

@@ -131,7 +131,7 @@ module Blazer
 
           if chart_type == "line"
             columns[1..-1].each_with_index.each do |k, i|
-              series << {name: k, data: rows.map{ |r| [r[0], r[i + 1]] }}
+              series << {name: k, data: rows.map { |r| [r[0], r[i + 1]] }}
             end
           else
             rows.group_by { |r| v = r[1]; (smart_values[columns[1]] || {})[v.to_s] || v }.each_with_index.map do |(name, v), i|

data/lib/blazer/statement.rb

@@ -33,9 +33,9 @@ module Blazer
           end
 
           unless value.is_a?(ActiveSupport::TimeWithZone)
-            if value =~ /\A\d+\z/
+            if value.match?(/\A\d+\z/)
               value = value.to_i
-            elsif value =~ /\A\d+\.\d+\z/
+            elsif value.match?(/\A\d+\.\d+\z/)
               value = value.to_f
             end
           end

data/lib/blazer/version.rb

@@ -1,3 +1,3 @@
 module Blazer
-  VERSION = "3.0.6"
+  VERSION = '3.0.8'
 end

data/lib/blazer_xlsx.rb

@@ -0,0 +1 @@
+require "blazer"

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: blazer_xlsx
 version: !ruby/object:Gem::Version
-  version: 3.0.6
+  version: 3.0.8
 platform: ruby
 authors:
 - Caio Garcia
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2024-03-22 00:00:00.000000000 Z
+date: 2024-08-21 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: railties
@@ -94,8 +94,8 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
-description: 
-email: 
+description:
+email:
 executables: []
 extensions: []
 extra_rdoc_files: []
@@ -220,6 +220,7 @@ files:
 - lib/blazer/slack_notifier.rb
 - lib/blazer/statement.rb
 - lib/blazer/version.rb
+- lib/blazer_xlsx.rb
 - lib/generators/blazer/install_generator.rb
 - lib/generators/blazer/templates/config.yml.tt
 - lib/generators/blazer/templates/install.rb.tt
@@ -249,7 +250,7 @@ files:
 homepage: https://github.com/CaioGarcia1/blazer_xlsx
 licenses: []
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -264,8 +265,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.5.6
-signing_key: 
+rubygems_version: 3.4.1
+signing_key:
 specification_version: 4
 summary: Blazer fork that adds support for exporting to XLSX
 test_files: []