Skip to main content

Shepherd writeup
draft-ietf-tls-svcb-ech

# Document Shepherd Write-Up for Group Documents

*This version is dated 4 July 2022.*

Thank you for your service as a document shepherd. Among the responsibilities is
answering the questions in this write-up to give helpful context to Last Call
and Internet Engineering Steering Group ([IESG][1]) reviewers, and your
diligence in completing it is appreciated. The full role of the shepherd is
further described in [RFC 4858][2]. You will need the cooperation of the authors
and editors to complete these checks.

Note that some numbered items contain multiple related questions; please be sure
to answer all of them.

## Document History

1. Does the working group (WG) consensus represent the strong concurrence of a
   few individuals, with others being silent, or did it reach broad agreement?

This I-D represents the consensus on the WG.

Please note that the text in this I-D was initially developed in the DNSOP WG,
went through IETF LC, and IESG review. The result of the IESG review was to take
the text in this I-D out of RFC 9460 (was draft-ietf-dnsop-svcb-http) and run the
new I-D through the TLS WG. The text in this I-D is essentially the same text
taken from -11 of draft-ietf-dnsop-svcb-http. In some respects, you could claim
that this I-D has consensus from multiple WGs.

2. Was there controversy about particular points, or were there decisions where
   the consensus was particularly rough?

There are two things that might be considered controversial and I am stretching here:
How this I-D arrived at TLS. Ben provided a good explanation here:
https://mailarchive.ietf.org/arch/msg/tls/Vct8iUc4IgSHENX2r9IGOQFLkyk/
ECH itself, but that seems to have subside.

3. Has anyone threatened an appeal or otherwise indicated extreme discontent? If
   so, please summarize the areas of conflict in separate email messages to the
   responsible Area Director. (It should be in a separate email because this
   questionnaire is publicly available.)

There have been no threats of appeal.

4. For protocol documents, are there existing implementations of the contents of
   the document? Have a significant number of potential implementers indicated
   plans to implement? Are any existing implementations reported somewhere,
   either in the document itself (as [RFC 7942][3] recommends) or elsewhere
   (where)?

This specification is implemented today by Chrome, Firefox, and Safari [1],
and is deployed on all Cloudflare free tier domains [2].

[1] https://chromestatus.com/feature/6196703843581952
[2] https://blog.cloudflare.com/announcing-encrypted-client-hello


## Additional Reviews

5. Do the contents of this document closely interact with technologies in other
   IETF working groups or external organizations, and would it therefore benefit
   from their review? Have those reviews occurred? If yes, describe which
   reviews took place.

As noted earlier, this I-D is about DNS service records and I’d argue there is
not a lot of expertise in the TLS WG on DNS, but the authors are experts. We
also made sure to get an early review from the DNSDIR; see
https://datatracker.ietf.org/doc/review-ietf-tls-svcb-ech-01-dnsdir-early-lemon-2024-03-29/.
The DNSDIR comments were discussed and resolved; see
https://mailarchive.ietf.org/arch/msg/tls/VQcqUuOXSE8sgcp4_CHa6Jlc394/.

Again, the text of this I-D is largely unchanged from what went through
IETF LC and IESG review as draft-ietf-dnsop-svcb-http before being separated
from what became RFC 9460. It was thoroughly reviewed at that point, and
modifications since then have been minimal.

6. Describe how the document meets any required formal expert review criteria,
   such as the MIB Doctor, YANG Doctor, media type, and URI type reviews.

Note that the underlying protocol (ECH) has undergone security analysis as documented
in https://www.cs.ox.ac.uk/people/vincent.cheval/publis/BCW-ccs22.pdf.

7. If the document contains a YANG module, has the final version of the module
   been checked with any of the [recommended validation tools][4] for syntax and
   formatting validation? If there are any resulting errors or warnings, what is
   the justification for not fixing them at this time? Does the YANG module
   comply with the Network Management Datastore Architecture (NMDA) as specified
   in [RFC 8342][5]?

N/A

8. Describe reviews and automated checks performed to validate sections of the
   final version of the document written in a formal language, such as XML code,
   BNF rules, MIB definitions, CBOR's CDDL, etc.

N/A

## Document Shepherd Checks

9. Based on the shepherd's review of the document, is it their opinion that this
   document is needed, clearly written, complete, correctly designed, and ready
   to be handed off to the responsible Area Director?

The Shepherd believes that this document is ready to go.

10. Several IETF Areas have assembled [lists of common issues that their
	reviewers encounter][6]. For which areas have such issues been identified
	and addressed? For which does this still need to happen in subsequent
	reviews?

Note that text in this I-D has already been through IETF LC.

11. What type of RFC publication is being requested on the IETF stream ([Best
	Current Practice][12], [Proposed Standard, Internet Standard][13],
	[Informational, Experimental or Historic][14])? Why is this the proper type
	of RFC? Do all Datatracker state attributes correctly reflect this intent?

Standards Track is requested. While the registry does not require Standards Track,
Standards Track is appropriate for this I-D because: It was ripped from a Standards
Track RFC, i.e., it would have been in RFC 9460 except for timing. There is sufficient
interest from implementers in supporting ECH with SVCB. 

12. Have reasonable efforts been made to remind all authors of the intellectual
	property rights (IPR) disclosure obligations described in [BCP 79][7]? To
	the best of your knowledge, have all required disclosures been filed? If
	not, explain why. If yes, summarize any relevant discussion, including links
	to publicly-available messages when applicable.

The Shepherd has confirmed with the authors that all required disclosures have been
filed.

13. Has each author, editor, and contributor shown their willingness to be
	listed as such? If the total number of authors and editors on the front page
	is greater than five, please provide a justification.

The Shepherd has confirmed with the authors that all are willing to be an author.

14. Document any remaining I-D nits in this document. Simply running the [idnits
	tool][8] is not enough; please review the ["Content Guidelines" on
	authors.ietf.org][15]. (Also note that the current idnits tool generates
	some incorrect warnings; a rewrite is underway.)

-04 had some nits, but -05 addresses them.

15. Should any informative references be normative or vice-versa? See the [IESG
	Statement on Normative and Informative References][16].

There are not many references and they seem to be categorized correctly.

16. List any normative references that are not freely available to anyone. Did
	the community have sufficient access to review any such normative
	references?
N/A

17. Are there any normative downward references (see [RFC 3967][9] and [BCP
	97][10]) that are not already listed in the [DOWNREF registry][17]? If so,
	list them.

This I-D includes a reference to draft-ietf-tls-esni, but this I-D will progress
with that I-D so the references will be resolved as part of the cluster the RFC
editor will create.

18. Are there normative references to documents that are not ready to be
	submitted to the IESG for publication or are otherwise in an unclear state?
	If so, what is the plan for their completion?

See q17.

19. Will publication of this document change the status of any existing RFCs? If
	so, does the Datatracker metadata correctly reflect this and are those RFCs
	listed on the title page, in the abstract, and discussed in the
	introduction? If not, explain why and point to the part of the document
	where the relationship of this document to these other RFCs is discussed.

N/A

20. Describe the document shepherd's review of the IANA considerations section,
	especially with regard to its consistency with the body of the document.
	Confirm that all aspects of the document requiring IANA assignments are
	associated with the appropriate reservations in IANA registries. Confirm
	that any referenced IANA registries have been clearly identified. Confirm
	that each newly created IANA registry specifies its initial contents,
	allocations procedures, and a reasonable name (see [RFC 8126][11]).

The authors of this I-D created the registry that this I-D registers values in so
there is little chance we got this wrong. The Shepherd double checked that the 5
columns needed for the registry are included and appropriate for this registration.

21. List any new IANA registries that require Designated Expert Review for
	future allocations. Are the instructions to the Designated Expert clear?
	Please include suggestions of designated experts, if appropriate.

N/A

[1]: https://www.ietf.org/about/groups/iesg/
[2]: https://www.rfc-editor.org/rfc/rfc4858.html
[3]: https://www.rfc-editor.org/rfc/rfc7942.html
[4]: https://wiki.ietf.org/group/ops/yang-review-tools
[5]: https://www.rfc-editor.org/rfc/rfc8342.html
[6]: https://wiki.ietf.org/group/iesg/ExpertTopics
[7]: https://www.rfc-editor.org/info/bcp79
[8]: https://www.ietf.org/tools/idnits/
[9]: https://www.rfc-editor.org/rfc/rfc3967.html
[10]: https://www.rfc-editor.org/info/bcp97
[11]: https://www.rfc-editor.org/rfc/rfc8126.html
[12]: https://www.rfc-editor.org/rfc/rfc2026.html#section-5
[13]: https://www.rfc-editor.org/rfc/rfc2026.html#section-4.1
[14]: https://www.rfc-editor.org/rfc/rfc2026.html#section-4.2
[15]: https://authors.ietf.org/en/content-guidelines-overview
[16]: https://www.ietf.org/about/groups/iesg/statements/normative-informative-references/
[17]: https://datatracker.ietf.org/doc/downref/
Back