strongmind-platform-sdk 3.21.1 → 3.21.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 57b1e8c4260deda607fb5e9dcf977c3a94f9c1571686ebe1d32dcb253bd22eb5
-  data.tar.gz: a1149d7e49f28bc4e4d8361dd9f193d78202406faed55c2e4e8bbe843c92f911
+  metadata.gz: efa1b649bf5efa1b5b2becd402edd82a48f3c56cfc138c4c6e48190b30c4c1ea
+  data.tar.gz: 8ec2206a55dc2ba2ed803509a3686761dcb544fb6c25b91d668dad8910fea3dd
 SHA512:
-  metadata.gz: b1191a72e8233bf5dc670e8f67ba3d3ba85c5798af7dbd1f6e7b9145e2e76815c78b7c70a29aa78fce90400dc55ad51bbb76406f779009bc2185f23d5611c988
-  data.tar.gz: 728aa199f77a5dd224e36f55152880d878e72720d3571e7cae87833a1ee909d85af966749cd74fbc183190c28b7e1b22d14301459dce81edb48a1cb8ea3221ce
+  metadata.gz: e97d871efc15d00d8508e9a0cc6dca74fcfa8a390385b18285253e0e50b9224fa9596f10c4ce576abf40329adf90f73df37c9db2c061345269e271bad25c5173
+  data.tar.gz: fc0e5681fecc25c5eef70ae0873511a70b7cd05d970d133464a37353f13d6d4efd1ce43484a792ba25fbbfb493d353fd5e87b4d899b4c6621987f312cfc3ece0

data/lib/platform_sdk/active_record/data_pipelineable.rb

@@ -13,15 +13,18 @@ module PlatformSdk
       end
 
       def send_pipeline_create
-        send_to_pipeline('created')
+        Jobs::SendNounToPipelineJob.perform_later(pipeline_payload('created'),
+                                                  one_roster_pipeline_payload)
       end
 
       def send_pipeline_update
-        send_to_pipeline('modified')
+        Jobs::SendNounToPipelineJob.perform_later(pipeline_payload('modified'),
+                                                  one_roster_pipeline_payload)
       end
 
       def send_pipeline_destroy
-        send_to_pipeline('destroyed')
+        Jobs::SendNounToPipelineJob.perform_later(pipeline_payload('destroyed'),
+                                                  one_roster_pipeline_payload(deleted: true))
       end
 
       def pipeline_payload(action)
@@ -67,20 +70,6 @@ module PlatformSdk
       def one_roster_pipeline_payload(deleted: false)
         {}
       end
-
-      def send_to_pipeline(action)
-        credentials = {
-          pipeline_host: ENV.fetch('DATA_PIPELINE_HOST', 'stage-di-data-pipeline-api.strongmind.com'),
-          pipeline_username: ENV.fetch('DATA_PIPELINE_USERNAME', 'canvas_prod'),
-          pipeline_password: ENV.fetch('DATA_PIPELINE_PASSWORD', '')
-        }
-        client = PlatformSdk::DataPipeline::Client.new(credentials)
-
-        client.post(pipeline_payload(action)) unless Rails.env.development?
-        return unless respond_to?(:one_roster_data_type) && !Rails.env.development? && one_roster_pipeline_payload(deleted: action == "destroyed").present?
-
-        client.post(one_roster_pipeline_payload(deleted: action == "destroyed"))
-      end
     end
   end
 end

data/lib/platform_sdk/canvas_api/client.rb

@@ -552,7 +552,7 @@ module PlatformSdk
             error_messages << "#{error[:message]} "
           end
         end
-        "#{message}: {#{error_messages}"
+        "#{message}: #{error_messages}"
       end
 
       def handle_rate_limiting(response)
@@ -568,10 +568,17 @@ module PlatformSdk
       # @param response [Faraday::Response]
       # @param message [String]
       def handle_post_response_errors(response, message)
-        result = MultiJson.load response.body, symbolize_keys: true
-        raise LockedModuleItemError, "#{message}: #{result[:message]}" if response.status == 403 and result[:message] == 'The module item is locked.'
-        raise MissingModuleItemError, "#{message}: #{result[:message]}" if response.status == 404
-        raise CanvasClientError, parse_response_errors(message:, result:)
+        message = "#{message},\n" + "host: #{host},\n" + "response: #{response.body}"
+        case response.status
+        when 403
+          raise LockedModuleItemError, message
+        when 404
+          raise MissingModuleItemError, message
+        when 401
+          raise UnauthorizedError, message
+        else
+          raise CanvasClientError, message
+        end
       end
     end
 
@@ -580,5 +587,6 @@ module PlatformSdk
     class LockedModuleItemError < CanvasClientError; end
 
     class MissingModuleItemError < CanvasClientError; end
+    class UnauthorizedError < CanvasClientError; end
   end
 end

data/lib/platform_sdk/jobs/send_noun_to_pipeline_job.rb

@@ -0,0 +1,31 @@
+# frozen_string_literal: true
+
+require 'active_job'
+
+module PlatformSdk
+  module Jobs
+    class SendNounToPipelineJob < ActiveJob::Base
+      queue_as :default
+
+      def perform(message, one_roster_message)
+        send_to_pipeline(message, one_roster_message)
+      end
+
+      private
+
+      def send_to_pipeline(message, one_roster_message)
+        credentials = {
+          pipeline_host: ENV.fetch('DATA_PIPELINE_HOST', 'stage-di-data-pipeline-api.strongmind.com'),
+          pipeline_username: ENV.fetch('DATA_PIPELINE_USERNAME', 'canvas_prod'),
+          pipeline_password: ENV.fetch('DATA_PIPELINE_PASSWORD', '')
+        }
+        client = PlatformSdk::DataPipeline::Client.new(credentials)
+
+        client.post(message) unless Rails.env.development?
+        return unless one_roster_message && !Rails.env.development?
+
+        client.post(one_roster_message)
+      end
+    end
+  end
+end

data/lib/platform_sdk/spec_support/shared_examples/data_pipelineable_examples.rb

@@ -1,6 +1,8 @@
 module PlatformSdk
   module SpecSupport
     RSpec.shared_examples 'DataPipelineable' do |params = {}|
+      include ActiveJob::TestHelper
+
       let(:shared_params) { params }
       let(:record) { build(described_class.to_s.underscore.to_sym) }
       let(:data_pipeline_client) { double(PlatformSdk::DataPipeline::Client) }
@@ -16,7 +18,9 @@ module PlatformSdk
       context 'create' do
 
         it 'posts a copy of our data to the data pipeline' do
-          record.save!
+          perform_enqueued_jobs do
+            record.save!
+          end
           expect(data_pipeline_client).to have_received(:post).once.with(hash_including(
                                                                            noun: described_class.pipeline_noun,
                                                                            data: record.pipeline_data('created'),
@@ -31,8 +35,10 @@ module PlatformSdk
       context 'destroy' do
 
         it 'posts a copy of our data to the data pipeline' do
-          record.save!
-          record.destroy
+          perform_enqueued_jobs do
+            record.save!
+            record.destroy
+          end
 
           expect(data_pipeline_client).to have_received(:post).once.with(hash_including(
                                                                            noun: described_class.pipeline_noun,
@@ -54,7 +60,9 @@ module PlatformSdk
           it 'posts a copy of our data to the data pipeline' do
             column = column_to_update(record)
             unless column.nil?
-              update_record(record, column)
+              perform_enqueued_jobs do
+                update_record(record, column)
+              end
               expect(data_pipeline_client).to have_received(:post).once.with(hash_including(
                                                                                noun: described_class.pipeline_noun,
                                                                                data: record.pipeline_data('modified'),
@@ -71,10 +79,12 @@ module PlatformSdk
           let(:action) { 'modified' }
 
           it 'does not post to the data pipeline' do
-            record.save!
-            column = column_to_update(record)
-            update_record(record, column)
-            record.save!
+            perform_enqueued_jobs do
+              record.save!
+              column = column_to_update(record)
+              update_record(record, column)
+              record.save!
+            end
             expect(data_pipeline_client).to have_received(:post).once.with(hash_including(
                                                                              noun: described_class.pipeline_noun,
                                                                              data: record.pipeline_data('modified'),

data/lib/platform_sdk/spec_support/shared_examples/one_roster_data_pipelineable_examples.rb

@@ -3,6 +3,8 @@
 module PlatformSdk
   module SpecSupport
     RSpec.shared_examples 'OneRosterDataPipelineable' do |record_keys|
+      include ActiveJob::TestHelper
+
       let(:data) do
         record_keys.each_with_object({}) do |key, hash|
           if key == '_type'
@@ -50,7 +52,9 @@ module PlatformSdk
         let(:action) { 'created' }
 
         it 'posts to the data pipeline' do
-          record.save!
+          perform_enqueued_jobs do
+            record.save!
+          end
           expect(data_pipeline_client).to have_received(:post).with(pipeline_payload)
         end
       end
@@ -68,8 +72,10 @@ module PlatformSdk
         end
 
         it 'posts to the data pipeline' do
-          record.save!
-          record.destroy
+          perform_enqueued_jobs do
+            record.save!
+            record.destroy
+          end
           expect(data_pipeline_client).to have_received(:post).with(pipeline_payload)
           expect(data_pipeline_client).to have_received(:post).with(pipeline_payload_deleted)
         end
@@ -86,7 +92,9 @@ module PlatformSdk
 
           it 'posts a copy of our data to the data pipeline' do
             column = column_to_update(record)
-            update_record(record, column)
+            perform_enqueued_jobs do
+              update_record(record, column)
+            end
             expect(data_pipeline_client).to have_received(:post).with(pipeline_payload)
           end
 
@@ -96,10 +104,12 @@ module PlatformSdk
           let(:action) { 'modified' }
 
           it 'does not post to the data pipeline' do
-            record.save!
-            column = column_to_update(record)
-            update_record(record, column)
-            record.save!
+            perform_enqueued_jobs do
+              record.save!
+              column = column_to_update(record)
+              update_record(record, column)
+              record.save!
+            end
             expect(data_pipeline_client).to have_received(:post).once.with(pipeline_payload)
           end
         end

data/lib/platform_sdk/version.rb

@@ -3,7 +3,7 @@
 module PlatformSdk
   MAJOR = 3
   MINOR = 21
-  PATCH = 1
+  PATCH = 3
 
   VERSION = "#{PlatformSdk::MAJOR}.#{PlatformSdk::MINOR}.#{PlatformSdk::PATCH}"
 end

data/lib/platform_sdk.rb

@@ -23,6 +23,7 @@ require "fog-aws"
 require "platform_sdk/asset_sync_initializer"
 require "platform_sdk/sidekiq/ecs_task_protection_middleware"
 require "platform_sdk/sidekiq"
+require "platform_sdk/jobs/send_noun_to_pipeline_job"
 
 module PlatformSdk
   class Error < StandardError; end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: strongmind-platform-sdk
 version: !ruby/object:Gem::Version
-  version: 3.21.1
+  version: 3.21.3
 platform: ruby
 authors:
 - Platform Team
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2024-11-12 00:00:00.000000000 Z
+date: 2024-11-25 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: faraday
@@ -273,6 +273,7 @@ files:
 - lib/platform_sdk/id_mapper/models/domain.rb
 - lib/platform_sdk/identity.rb
 - lib/platform_sdk/identity/clients.rb
+- lib/platform_sdk/jobs/send_noun_to_pipeline_job.rb
 - lib/platform_sdk/learnosity_api.rb
 - lib/platform_sdk/learnosity_api/client.rb
 - lib/platform_sdk/one_roster.rb