avalonmediasystem / avalon

Avalon Media System – Samvera Application
http://www.avalonmediasystem.org/
Apache License 2.0
93 stars 51 forks source link

Playlist LTI Share Links Not Working #5823

Closed joncameron closed 1 month ago

joncameron commented 5 months ago

Description

Error (after selecting the module item in Canvas with the LTI share link for the playlist)

Screenshot 2024-05-06 at 12 13 45 PM

Same Error for Inline Embed (not "open in new tab")

Screenshot 2024-05-29 at 12 11 14 PM

Done Looks Like

joncameron commented 5 months ago

Q: Does this also occur when we don't set "open in new tab"? A: Yes.

joncameron commented 4 months ago

Will require investigation; maybe something in Canvas that we missed. Seems unusual that other external URLs from Avalon get authenticated properly. What is it about this route?

cjcolvar commented 3 months ago

I tested this and found that the LTI share links work when added as an "External Tool" but not as an "External Url". Is it important that these work as "External Url" specifically or is "External Tool" sufficient? I think that the LTI params hash isn't being sent with "External Url" and that's why login fails.

elynema commented 3 months ago

@joncameron Do we need to update documentation to be clear that the URLs should be included in Canvas as "External Tool"? Or is this something that used to work and now isn't.

joncameron commented 3 months ago

I'm going to update our documentation as Emily suggested and close this issue.

joncameron commented 1 month ago

https://samvera.atlassian.net/wiki/x/AQBBn