Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Opencast needs to change the cookie settings #1085

Closed
tgloeggl opened this issue Nov 25, 2024 · 4 comments · Fixed by elan-ev/opencast_nginx#5
Closed

Opencast needs to change the cookie settings #1085

tgloeggl opened this issue Nov 25, 2024 · 4 comments · Fixed by elan-ev/opencast_nginx#5
Labels
type:feature v:3 Everything related to the Stud.IP Opencast Plugin Version 3.x

Comments

@tgloeggl
Copy link
Member

tgloeggl commented Nov 25, 2024

Opencast needs to facilitate

https://developer.mozilla.org/en-US/docs/Web/Privacy/Privacy_sandbox/Partitioned_cookies
https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/Set-Cookie#samesitesamesite-value

Without these changes, LTI WILL break soon completly and is already broken in Strict mode in Firefox! Currently only the folling (maybe iconmplete list) of features are affected:

  • Video upload
  • Courseware block

These need cookies to work in an iframe and therefore the Partitioned-Attribte and SameSite=None needs to be set for the cookies from Opencast.

Screenshot 2024-11-27 at 11-25-17 Screenshot_from_2024-11-26_12-45-30 png (PNG Image 3738 × 1845 pixels) — Scaled (46%)

@tgloeggl tgloeggl added v:3 Everything related to the Stud.IP Opencast Plugin Version 3.x type:feature labels Nov 25, 2024
@tgloeggl tgloeggl modified the milestones: Opencast 3.19, V3.20 Nov 25, 2024
@KatrinIhler
Copy link
Member

If this is something Opencast needs to do, does it make sense to create an issue there as well?

@tgloeggl tgloeggl modified the milestones: V3.20, V3.21 Nov 29, 2024
@pettobia
Copy link

pettobia commented Dec 3, 2024

What is the current status? @tgloeggl

@mtneug
Copy link

mtneug commented Dec 11, 2024

I preached this for ages about LTI and yes, Partitioned will be a solution. However, this is currently not implemented in Safari (as mentioned in the MDN Web Docs). See WebKit/standards-positions#50 for a discussion.

@tgloeggl
Copy link
Member Author

I will have to further investigate this issue regarding Safari, but without researching the reasons it a quick test it seems to just work in as of now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type:feature v:3 Everything related to the Stud.IP Opencast Plugin Version 3.x
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants