atomic_lti 1.5.6 → 1.5.7

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 81905bc42beff53064deaef814aa929b43f73766f81e2159739cfa187d23c7d7
-  data.tar.gz: 156ab5f22d91b0e9a74f101201f674bc416fcaa36ad4942812ffee11017d53da
+  metadata.gz: f5de79b035c9dcf52e0579701165c9d317724db6fad17ad072d767eb65ca6c2a
+  data.tar.gz: 30362e1c72ca07662455736e23a6a784539a732da6b4e23375f220ac87c74068
 SHA512:
-  metadata.gz: 2e19ed34c5951f99b201149f13dcda60099c3a3da3cb56046d9126cc28c966afc10fc1a06f949ba3fcae24304938bed25763ed653c9468dd8c672d1fb6182540
-  data.tar.gz: 3a516460e622b3d292027a69a7e6dc6fd03aaa693f7806182fc6a397680a79f9b8fce5783a0a72b9e8813da18eaf72041a744b2cec4a8486cb7b3f10ccd5b57f
+  metadata.gz: 5b7118809d65edc14c6692eca2ac35c42b9bff12a1b76a1f77674ae189a0422be118736c2ede3accce66150d6ca7cb4dfc1f8058569d31f03a055f0afb1d96b5
+  data.tar.gz: c414e39c0bb8c3e092fbabea946415afe405bde2bd6012b4a4fae5b7b49201a541f70a6116c71e44464dd94ca8f7f4562b007df77446bf47a322e6902d52ed96

data/lib/atomic_lti/open_id_middleware.rb

@@ -111,12 +111,26 @@ module AtomicLti
       target_link_uri = id_token_decoded[AtomicLti::Definitions::TARGET_LINK_URI_CLAIM] ||
         File.join("#{uri.scheme}://#{uri.host}", AtomicLti.default_deep_link_path)
 
+      # We want to strip out the redirect path params from the request params
+      # so that we can support having the redirect path be the same as the
+      # launch path, only differentiated by a query parameter. This is needed
+      # because socialize journals redirects straight to /lti_launches and we
+      # have preexisting lti keys that we won't get to migrate and can't break
+      redirect_uri = URI.parse(AtomicLti.oidc_redirect_path)
+      redirect_path_params = if redirect_uri.query
+                               CGI.parse(redirect_uri.query)
+                             else
+                               {}
+                             end
+
+      launch_params = request.params.except(*redirect_path_params.keys)
+
       html = ApplicationController.renderer.render(
         :html,
         layout: false,
         template: "atomic_lti/shared/redirect",
         assigns: {
-          launch_params: request.params,
+          launch_params: launch_params,
           launch_url: target_link_uri,
         },
       )
@@ -133,7 +147,7 @@ module AtomicLti
       redirect_path_params = if redirect_uri.query
                                CGI.parse(redirect_uri.query)
                              else
-                               []
+                               {}
                              end
 
       matches_redirect_path = request.path == redirect_uri.path

data/lib/atomic_lti/version.rb

@@ -1,3 +1,3 @@
 module AtomicLti
-  VERSION = "1.5.6".freeze
+  VERSION = "1.5.7".freeze
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: atomic_lti
 version: !ruby/object:Gem::Version
-  version: 1.5.6
+  version: 1.5.7
 platform: ruby
 authors:
 - Matt Petro
@@ -10,7 +10,7 @@ authors:
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-10-28 00:00:00.000000000 Z
+date: 2023-10-30 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: pg