-
Notifications
You must be signed in to change notification settings - Fork 130
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
Allow execution in third party contexts #446
Comments
Hi @nilols, In order to better understand your issue:
Thanks |
The
Yes, and with another domain name then the page currently visited |
Maybe this https://blog.chromium.org/2019/10/developers-get-ready-for-new.html explain the issue with |
OK, I did not realize that execution in an iFrame was considered as a third party context. We could probably allow this behavior through a dedicated configuration parameter, we'll discuss it internally. |
This is available in |
Hey, I have this issue too with the same use case - my company iframes are in other sites and im getting:
|
The fix has been release as part of |
tried to upgrade to 1.24.0 but still the same cookies warning and no data on the datadog website... Is this ability of iframe tracking is compatible with the datadogRUM SDK? |
@gevalter: By using using |
oh you right, changed to https and all works, thanks! |
Still seeing this issue: I have RUM running inside an iframe, seems like this should work as per #555 EDIT: Ah duh cross origin issues b/w localhost and my iframed app - this is only an issue in local dev. |
We want to use RUM in our plugin, the plugin is loaded into an iframe of another company's application, but the recent
samesite
attribute change, which I understand is needed, prevents us from using RUM, due to that the cookie test fails whensamesite=strict
I found this page that talks about the
samesite
attribute and third party contexts. And as a workaround I created my own RUM sdk version, were I instead ofsamesite=strict
usesamesite=none; secure
and that works fine, but it would be better if this was supported out of the box.The text was updated successfully, but these errors were encountered: