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

Anonymous iframe: credentials attribute #20

Closed
antosart opened this issue Jun 8, 2021 · 3 comments
Closed

Anonymous iframe: credentials attribute #20

antosart opened this issue Jun 8, 2021 · 3 comments

Comments

@antosart
Copy link

antosart commented Jun 8, 2021

What is the semantic of the credentials attribute? credentials=omit vs credentials=present doesn't look good because of inconsistency (omit is a verb, while present is an adjective). In case, I'd suggest having credentials=use instead.

However, I am a bit unhappy with credentials=omit since this is misleading. Credentials are partitioned, not omitted. Moreover, I don't see the point of having a non-boolean attribute for something that can only be set and not unset (credentials=use - if we go with it - has no point of existing, apart from giving a value to be returned by the getter, since a subframe of an anonymous iframe is always anonymous and this cannot be overwritten).

I would reconsider having a boolean attribute instead, something like <iframe anonymous> or <iframe partitioned> or <iframe isolated> or <iframe partitioned-storage> or <iframe clean-credentials> (I fear none of these is a particularly good choice though).

@ArthurSonzogni
Copy link
Contributor

Explainer updated from:

<iframe credentials=omit>

toward

<iframe anonymous>

in PR:
ArthurSonzogni@5d12c9a

So we can close this, I guess, once it is merged.

@ArthurSonzogni
Copy link
Contributor

@camillelamy: can we close this?

@camillelamy
Copy link
Owner

Done.

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

3 participants