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

Use of cookies and local storage for remembering registered user credentials #27

Open
eldanb opened this issue Nov 29, 2020 · 0 comments

Comments

@eldanb
Copy link

eldanb commented Nov 29, 2020

In section 3, FIDO re-authentication, it is stated that "If the user clicks on “Next”, then check whether you have a credential id associated with the user and device (for example, check a cookie or read from local storage)."

Two points that I think are worth addressing here:

  1. Safari, with ITP support, would actually delete local storage or JS-accessible cookies after 7 days without use. So if we recommend using either here -- need to make sure that implementers are aware that this will need to have some sort of fallback (what that fallback would be?). Alternatively suggest using HTTP-only cookies which aren't deleted after 7 days and possibly mirroring their values back to the client by the Server.

  2. On Windows different browsers (e.g Chrome and Edge) use the platform's FIDO2 client, which shares registered platform-attached credentials among applications. So if I go to https://www.example.com from Chrome and register a platform-attached credential, then I go to https://www.example.com from Edge -- that platform-attached credential should be made available to me. But since cookies / local storage aren't shared by browsers -- this won't be reflected to the user.
    We should, at a minimum, refer to this in this document -- not to mention work on the standard further to accommodate this use case.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant