Last Call Review of draft-ietf-manet-smf-mib-08
review-ietf-manet-smf-mib-08-secdir-lc-hanna-2013-10-24-00
Request | Review of | draft-ietf-manet-smf-mib |
---|---|---|
Requested revision | No specific revision (document currently at 13) | |
Type | Last Call Review | |
Team | Security Area Directorate (secdir) | |
Deadline | 2013-10-16 | |
Requested | 2013-10-03 | |
Authors | Robert Cole , Joseph P. Macker , Brian Adamson | |
I-D last updated | 2013-10-24 | |
Completed reviews |
Secdir Last Call review of -08
by Steve Hanna
(diff)
Secdir Telechat review of -10 by Steve Hanna (diff) Opsdir Telechat review of -10 by Jürgen Schönwälder (diff) |
|
Assignment | Reviewer | Steve Hanna |
State | Completed | |
Request | Last Call review on draft-ietf-manet-smf-mib by Security Area Directorate Assigned | |
Reviewed revision | 08 (document currently at 13) | |
Result | Has nits | |
Completed | 2013-10-24 |
review-ietf-manet-smf-mib-08-secdir-lc-hanna-2013-10-24-00
I have reviewed this document as part of the security directorate's ongoing effort to review all IETF documents being processed by the IESG. These comments were written primarily for the benefit of the security area directors. Document editors and WG chairs should treat these comments just like any other last call comments. While I am not an expert in SNMP, SMF, or MANET, I found this document to be well-written and easy to understand. More relevant to this review, the security of the protocol is adequate and the Security Considerations section is exemplary. I did notice two typos: * In the Security Considerations section, the commentary on smfConfiguredOpMode includes the words "this writable configuration objects define". This should end in "object define", I think. * In the Security Considerations section, the commentary on smfNhdpRssaMesgTLVIncluded includes the words "the the". Of course, that should be just "the". With these corrections, I think the document is ready to publish. Thanks, Steve