canvas_sync 0.17.41 → 0.17.42

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: cfa9a724751a0d163496133f4c928f1ccfca544413d668ef045bb30178dd37fc
-  data.tar.gz: c6107cc707e8474dc23e429760b276d0f91f5e482860cd870b4069279bcc9120
+  metadata.gz: 6637b97dd9b51f6d3babe9b022904d9e77b1c886e62707fb857db5caba909fc3
+  data.tar.gz: 1bfa80e171b9b92b12ebad2fdc732b558e59fe9271175ce7d1dd2680a81e2b0c
 SHA512:
-  metadata.gz: 6750a83429e8004c6c1a46079e349c63b5b14abd6ccc35f33117f74d791030708acd9918fe45f2a78f7da9bb0f07a5f3d2a1cd8990eabceb46541d430ae13719
-  data.tar.gz: 104ef9be9f7dab34e823f9705595ed4ee411b2ac1ef9a92fd04e4e42861997ccf7d85705765b65c3bad833bb25fa1409fb32354a2d816cd1034f05a94a0e90d1
+  metadata.gz: 0af624f526294b480e6b24814ab9156f3970056160eb59e0aafdd9af2849984ef34668099cce16f978257b6311c5fbfe69e2fe9e0bfd52224034c7e8018afbfc
+  data.tar.gz: 2d2d89dc832f3cb4d901641f00c930f929ca410f86b2890689909f43fea3fb29d56584fa72c5871382101d946ff7d50fde3d044375ae03d1b8de230bba13788b

data/lib/canvas_sync/engine.rb

@@ -8,6 +8,7 @@ module CanvasSync
     initializer "canvas_sync.safe_yaml_classes" do |app|
       app.config.active_record.yaml_column_permitted_classes ||= []
       app.config.active_record.yaml_column_permitted_classes |= [Symbol, ActiveSupport::HashWithIndifferentAccess]
+      ActiveRecord::Base.yaml_column_permitted_classes |= app.config.active_record.yaml_column_permitted_classes
     rescue
     end
 
@@ -40,12 +41,14 @@ module CanvasSync
 
       require 'panda_pal'
 
-      def canvas_sync_client(account_id = nil)
-        Bearcat::Client.new(
-          prefix: current_organization.canvas_url,
-          token: current_organization.canvas_api_token,
-          master_rate_limit: (Rails.env.production? && !!defined?(Redis) && ENV['REDIS_URL'].present?),
-        )
+      class ::Object
+        def canvas_sync_client(account_id = nil)
+          Bearcat::Client.new(
+            prefix: current_organization.canvas_url,
+            token: current_organization.canvas_api_token,
+            master_rate_limit: (Rails.env.production? && !!defined?(Redis) && ENV['REDIS_URL'].present?),
+          )
+        end
       end
     rescue LoadError
     end

data/lib/canvas_sync/jobs/begin_sync_chain_job.rb

@@ -6,6 +6,10 @@ module CanvasSync
       def perform(chain_definition, globals = {})
         @globals = globals
 
+        if globals[:updated_after] == nil && Rails.env.development?
+          globals[:updated_after] = false
+        end
+
         if globals[:updated_after] == false
           globals[:updated_after] = nil
         elsif !globals[:updated_after].present? || globals[:updated_after] == true

data/lib/canvas_sync/jobs/report_processor_job.rb

@@ -26,7 +26,7 @@ module CanvasSync
       def download(report_name, report_url)
         Dir.mktmpdir do |dir|
           file_path = "#{dir}/#{report_name}"
-          IO.copy_stream(open(report_url), file_path)
+          IO.copy_stream(URI.open(report_url), file_path)
           yield file_path
         end
       end

data/lib/canvas_sync/jobs/report_starter.rb

@@ -35,7 +35,11 @@ module CanvasSync
 
       protected
 
-      def merge_report_params(options, params={}, term_scope: true)
+      # Ruby 3 changed how kwargs are handled. _kw_placeholder allows for backwards compatibility
+      # In Ruby 2, merge_report_params(options, params) would parse as merge_report_params(options, params={}, **params) (wtf?), so
+      # merge_report_params(options, params, {}) is used. That doesn't work in Ruby 3.
+      # In order to maintain compatibility with 2 and with any apps, this oddness is needed
+      def merge_report_params(options, params={}, _kw_placeholder=nil, term_scope: true)
         term_scope = options[:canvas_term_id] || batch_context[:canvas_term_id] if term_scope == true
         if term_scope.present?
           params[:enrollment_term_id] = term_scope

data/lib/canvas_sync/version.rb

@@ -1,3 +1,3 @@
 module CanvasSync
-  VERSION = "0.17.41".freeze
+  VERSION = "0.17.42".freeze
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: canvas_sync
 version: !ruby/object:Gem::Version
-  version: 0.17.41
+  version: 0.17.42
 platform: ruby
 authors:
 - Nate Collings
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2023-01-12 00:00:00.000000000 Z
+date: 2023-01-24 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -276,6 +276,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: open-uri
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: activejob
   requirement: !ruby/object:Gem::Requirement