Last Call Review of draft-ietf-netconf-tls-client-server-03
review-ietf-netconf-tls-client-server-03-yangdoctors-lc-bierman-2017-07-28-00
| Request | Review of | draft-ietf-netconf-tls-client-server-03 |
|---|---|---|
| Requested revision | 03 (document currently at 41) | |
| Type | Last Call Review | |
| Team | YANG Doctors (yangdoctors) | |
| Deadline | 2017-07-31 | |
| Requested | 2017-07-10 | |
| Requested by | Mehmet Ersue | |
| Authors | Kent Watsen | |
| I-D last updated | 2017-07-28 | |
| Completed reviews |
Yangdoctors Last Call review of -03
by Andy Bierman
(diff)
Yangdoctors Last Call review of -24 by Andy Bierman (diff) Secdir Last Call review of -25 by Watson Ladd (diff) |
|
| Assignment | Reviewer | Andy Bierman |
| State | Completed | |
| Request | Last Call review on draft-ietf-netconf-tls-client-server by YANG Doctors Assigned | |
| Reviewed revision | 03 (document currently at 41) | |
| Result | Ready w/nits | |
| Completed | 2017-07-28 |
review-ietf-netconf-tls-client-server-03-yangdoctors-lc-bierman-2017-07-28-00
Review: draft-ietf-netconf-tls-client-server-03 Modules: (M1) ietf-tls-client@2017-06-13.yang (M2) ietf-tls-common@2017-06-13.yang (M3) ietf-tls-server@2017-06-13.yang YANG Usage: I did not find anything wrong in either module. pyang and yangdump-pro do not report any errors or warnings. Comments: C1: (M2) identity tls-1.2 Why are there no identities defined for tls-1.0 and tls-1.1? C2: Sec 1.2 Tree Diagrams Old text should be replaced with reference to draft-ietf-netmod-yang-tree-diagrams-01 C3: (M1) IETF copyright says 2014; change to 2017 (M3) IETF copyright says 2014; change to 2017