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

[RFC6265bis, Editorial]: Define "public suffix" in the terminology section. #1038

Merged
merged 3 commits into from
Jan 28, 2020

Conversation

mikewest
Copy link
Member

Closes #1033.

@mikewest mikewest requested a review from reschke January 22, 2020 09:23
@mikewest
Copy link
Member Author

Would you mind taking a look, @reschke?

Copy link
Contributor

@reschke reschke left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

that certainly fixes the original issue; I wonder whether there's a way to be more precise but it seems in the end it's what the list says, right? It probably would be good if we could say that the Mozilla list is somehow normative. Is it?

@mikewest
Copy link
Member Author

I think this document has the conceptual conceit that it might be possible to have distinct understandings of public suffixes. Browsers certainly have coalesced around https://publicsuffix.org/, but it's not clear to me that it's "normative" in any sense other than practice. A de facto standard, not de jure.

I'm happy to just say "Use the PSL", as that's most likely to be most compatible?

@reschke
Copy link
Contributor

reschke commented Jan 22, 2020

Well, if we have normative requirements attached to whether something is a public suffix, then we need to define how to decide that.

If there's a risk that the list ever moves, we could introduce an indirection through a new IANA registry (just holding the location of the public list, not the list itself).

@mikewest
Copy link
Member Author

The current normative requirements boil down to "Hey, user agent. Go figure out which suffixes you think are public. The PSL might be helpful. Have fun!" I'm happy to make that suggestion normative by bringing in the PSL in the same way URL does (https://url.spec.whatwg.org/#host-public-suffix). Would you be happy with that outcome?

@reschke
Copy link
Contributor

reschke commented Jan 22, 2020

Yes, this seems to be needed (maybe not in the same words, but...).

"a domain that is controlled by a public registry", and are also known as
"effective top-level domains" (eTLDs). For example, `site.example`'s public
A "public suffix" is a domain that is controlled by a public registry, such as
"com", "co.uk", and "pvt.k12.wy.us". For example, `site.example`'s public
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would it be worth making a comparison to "delegation-centric zone" from RFC 8499? It's certainly possible for a public suffix to not be delegation-centric (e.g., github.io) and there are some delegation-centric zones that are not public suffixes (most notably the DNS root zone itself), but there is nevertheless substantial overlap in both concept (different authority between parent and child domain) and contents.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not sure that bringing another concept in would clarify, especially given the discrepancies you note between "delegation-centric zone" and existing user agent understanding of "public suffix". I think I'd prefer to continue using the term that user agents have generally applied to cookies.

@mikewest
Copy link
Member Author

@reschke I've update the patch to more explicitly point to the PSL's algorithm, and switched from "registered domain" to "registrable domain" to match URL. WDYT?

@reschke
Copy link
Contributor

reschke commented Jan 28, 2020

Works for me...

@mikewest mikewest merged commit c4f6109 into master Jan 28, 2020
@mikewest mikewest deleted the psl branch January 28, 2020 12:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

6265bis: public suffix def
4 participants