-
Notifications
You must be signed in to change notification settings - Fork 25
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
Fixing 3rd party cookie issues by supporting LTI OIDC Login flow with LTI Client Side postMessages #129
Comments
We have had to address this as well. I am not a fan of IMS Globals solution for this. It was not very well thought out. To me, the answer is CHIPs or Partitioned Cookies. You can read more about that here. https://developer.mozilla.org/en-US/docs/Web/Privacy/Partitioned_cookies Firefox already partitions all cookies by default, and chrome is killing 3rd party cookies support slowly this year. Partitioned cookies allow the same behavior you have always expected. The one caveat to this is with Safari. But even safari has a solution. We built our own Safari solution taking advantage of the |
Dear @erikdonohoo , thanks a lot, that is very helpful! |
@erikdonohoo did you need to do anything special with this library to get that solution working? I still need to read up on the links shared by @phaazebroek, but if there are improvements we can make to this library to help support / accommodate these changes, I'd be open to it. |
^^ I'll echo what Erik said. Nothing special is needed other than jumping through the hoops the various browsers put in place. Safari needing the most work. In PHP-land (relevant to this library) the opt-in attribute needed for CHIPS support ('Partitioned') isn't supported by core PHP yet either: See php/php-src#12646 |
@erikdonohoo Do you have any pointers or examples to share on how you implemented to Safari API? First time I'm running into this issue and I'm a little lost. Managed the Samesite/Secure workaround but would much prefer to avoid the postMessage solution. |
Thanks for you all your work with this library!!!
Recently browsers have become more strict with regard to 3rd party cookies, with consequences for LTI 1.3 tools within iframes not being loaded: https://www.unicon.net/insights/blogs/the-solution-for-third-party-cookies-in-lti-1.3-has-arrived
There is now an approach using postmassage that seems to be supported by D2L and blackboard:
https://www.imsglobal.org/spec/lti-cs-oidc/v0p1#problem-description-blocked-cookies-prevent-proper-oidc-flow
Would that be something you would consider implementing in this library?
Thanks!
The text was updated successfully, but these errors were encountered: