panda_pal 5.6.2 → 5.6.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a24c53564f5782a10345c204de6908a094821e30dc7e0803ef7233d1ec5c9f1a
-  data.tar.gz: ac8f6d55d565ba718fbb3359d7ae129f346b891db9f35be0560a077587ae8812
+  metadata.gz: 79e31ac9d48be7e13d818cd57950034fa982726721bb9c451bac65ee3aaee702
+  data.tar.gz: 1bf00f07eba7546e160b2ee268e3bbfd37fdd3dfdf339999e8f322c43ceb6780
 SHA512:
-  metadata.gz: 0b9d02c9710ea8599c3a1e207c39a5f0540ec2669c50555a5da3f84a35a13e8856eeb30157bc837e94fc8e5d858de96c4b575c2a638c99b1dffa83a7376260af
-  data.tar.gz: 38aca64711cb7659b44c8b196b5813ef9c0d1f189bbc0f36a10cc0a17cee559743d83cd01151c952dbb984d12141febefcb179647167acb57f268a2b608f6c64
+  metadata.gz: 4dfa6dbf1aff76385844bb204aab7298d4096410df44cc2646643fe28bd2a2d54c9960f932c624544ee990218fc95022e27964c73b7604545fafa6618630fa0b
+  data.tar.gz: 0a74434217765714b28247315515ba9c6c71931d367e96130d3ed51a4f334ce43c03d80a0bdb6dfeee27caeff98f36c857cd525e61ee294a36db88f2619e4e76

data/app/controllers/panda_pal/lti_v1_p3_controller.rb

@@ -12,6 +12,13 @@ module PandaPal
     around_action :switch_tenant, only: [:resource_link_request]
 
     def login
+      # TODO Figure out how to make this support deployment_ids
+      #   May need to move Platform#jwks_url and Platform#authentication_redirect_url to
+      #   a new GenericPlatform class/model and then associate the here-generated Session with
+      #   the public tenant, re-homing it once we have a deployment_id available
+      #   ... or create a new AuthState model instead of using Session here
+      @organization = PandaPal::Organization.find_by!(key: params[:client_id])
+
       current_session_data[:lti_oauth_nonce] = SecureRandom.uuid
 
       @form_action = current_lti_platform.authentication_redirect_url

data/app/models/panda_pal/platform.rb

@@ -3,6 +3,8 @@ begin
 rescue LoadError
 end
 
+require 'httparty'
+
 module PandaPal
   class Platform
     def public_jwks
@@ -49,7 +51,7 @@ module PandaPal
       "#{base_url}/login/oauth2/token"
     end
 
-    def base_url; org.canvas_url; end
+    def base_url; @organization.canvas_url; end # TODO Dissolve?
 
     module OrgExtension
       def install_lti(host: nil, context: :root_account, version: 'v1p1', exists: :error)

data/lib/panda_pal/helpers/controller_helper.rb

@@ -59,7 +59,15 @@ module PandaPal::Helpers
       raise JSON::JWT::VerificationFailed, 'error decoding id_token' if decoded_jwt.blank?
 
       client_id = decoded_jwt['aud']
-      @organization = PandaPal::Organization.find_by!(key: client_id)
+      deployment_id = decoded_jwt['https://purl.imsglobal.org/spec/lti/claim/deployment_id']
+
+      if deployment_id.present?
+        @organization ||= PandaPal::Organization.find_by(key: "#{client_id}/#{deployment_id}")
+        @organization ||= PandaPal::Organization.find_by(key: deployment_id)
+      end
+
+      @organization ||= PandaPal::Organization.find_by(key: client_id)
+
       raise JSON::JWT::VerificationFailed, 'Unrecognized Organization' unless @organization.present?
 
       decoded_jwt.verify!(current_lti_platform.public_jwks)
@@ -124,8 +132,6 @@ module PandaPal::Helpers
 
     def organization_key
       org_key ||= params[:oauth_consumer_key]
-      org_key ||= "#{params[:client_id]}/#{params[:deployment_id]}" if params[:client_id].present? && params[:deployment_id].present?
-      org_key ||= params[:client_id] if params[:client_id].present?
       org_key ||= session[:organization_key]
       org_key
     end

data/lib/panda_pal/helpers/route_helper.rb

@@ -10,10 +10,7 @@ module PandaPal::Helpers::RouteHelper
 
     lti_options = options.delete(:lti_options) || {}
     lti_options[:auto_launch] = options.delete(:auto_launch)
-
-    if lti_options[:auto_launch].nil?
-      lti_options[:auto_launch] = (@scope[:path] || '').include?(':organization_id')
-    end
+    lti_options[:auto_launch] = true if lti_options[:auto_launch].nil? && (@scope[:path] || '').include?(':organization_id')
 
     lti_options[:route_helper_key] = path.split('/').reject(&:empty?).join('_')
     post(path, options.dup, &block)

data/lib/panda_pal/version.rb

@@ -1,3 +1,3 @@
 module PandaPal
-  VERSION = "5.6.2"
+  VERSION = "5.6.3"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: panda_pal
 version: !ruby/object:Gem::Version
-  version: 5.6.2
+  version: 5.6.3
 platform: ruby
 authors:
 - Instructure ProServe
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2022-08-31 00:00:00.000000000 Z
+date: 2022-09-01 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails