-
Notifications
You must be signed in to change notification settings - Fork 263
with 3rd party cookies disabled, primary IdP flow fails inexplicably #3520
Comments
Our first problem, is how can we possible know that 3rd party cookies are disabled? Detection is non-trivial. Here's some noodling from the depths of IRC:
|
@lloyd related: #3220 and a branch I've got cooking to improve localstorage detection: jaredhirsch/browserid@dev...issue_3220_improve_localstorage_detection |
@6a68 right! relationship is that detecting broken local storage will help us detect blocked 3rd party cookies, because the two things tend to be disabled in lockstep? |
This occurs with firefox when 3rd party cookies is set to "never". Or with chrome when "Block third-party cookies and site data" is selected. It's interesting to consider user facing messaging vs. just making it work, which would take a creative, tricky, but small protocol change. |
I am amazed no issue has been filed about this before, we have been discussing it as a problem for at least 6 months! |
Similar, but not exact - #3169 |
An approach similar to what the communication iframe does can be used:
|
rather detecting and messaging, I'l like to see if we can actually solve the underlying problem. |
It seems terrible that we'd need to message people that if they want to use
|
So, we've rolled out IdP for @mozilla.com users now, and hit this right away. I realize that ideally we'd like to solve this altogether, but in the interim I'd like to have some sensible user feedback about what the problem is. Right now it fails in a completely inexplicable (and highly frustrating) manner. |
+1. Can we stand up a domain and try to set a third party cookie with it as part of the dialog's bootup? |
could we with each our of idps (yahoo bridge, gmail bridge, mozidp), include a simple cookie check on the provisioning page, and if cookies don't stick, raiseProvisioningError('no 3rd party cookies')? |
I'd rather do this once at the shim level, instead of N times at N providers, if at all possible... |
@callahad - #3520 (comment) - same approach I was thinking about. Unfortunately, the shim isn't the place to do it because that won't take care of detecting disabled 3rd party cookies for pseudo-native implementations like FirefoxOS. As an alternative, we can embed 3rd party iframe in the dialog to do the check. |
@shane-tomlinson Sorry, that's what I meant -- the popup hosted at login.persona.org could embed an iframe to persona-cookie-check.org or similar. |
Because of differing browser behavior, I worry about the subtleties here. the feature we require is for a cookie set first party, to be readable third party. If we want to do a check and better messaging (rather than a protocol change to fix the underlying problem) I think it needs to be extremely similar to the feature that we need. Basically positing that the various checkboxes that browser's offer around "3rd party cookies", impose different restrictions. On Jul 18, 2013, at 6:47 AM, Dan Callahan notifications@github.com wrote:
|
* This is a standin for checking for disabled 3rd party cookies. If we expect provisioning to succeed, but it fails, this is probably due to disabled 3rd party cookies. fixes #3520
* This is a standin for checking for disabled 3rd party cookies. If we expect provisioning to succeed, but it fails, this is probably due to disabled 3rd party cookies. fixes #3520
* This is a standin for checking for disabled 3rd party cookies. If we expect provisioning to succeed, but it fails, this is probably due to disabled 3rd party cookies. fixes #3520
Link to #2079 |
Is there a plan to get rid of the need for 3rd party cookies altogether? So that users who block all third party cookies don't need to add an exception? |
Yes! A small change to the idp protocol will allow users with 3rd party cookies completely disabled to use persona. This user will need to authenticate more frequently. TanviHacks notifications@github.com wrote: Is there a plan to get rid of the need for 3rd party cookies altogether? So that users who block all third party cookies don't need to add an exception? Reply to this email directly or view it on GitHub: |
I think the plan goes like this:
|
The failure mode is confusing. We need to better message this to users, especially with gmail coming. labeling 4 star to spark discussion. It seems like there are a lot of issues around 3rd party cookies opened and closed, but none seem to address the primary idp case and get us a satisfying user facing error message.
The text was updated successfully, but these errors were encountered: