Skip to main content

Telechat Review of draft-ietf-netext-wifi-epc-eap-attributes-09
review-ietf-netext-wifi-epc-eap-attributes-09-genart-telechat-yee-2014-08-07-00

Request Review of draft-ietf-netext-wifi-epc-eap-attributes
Requested revision No specific revision (document currently at 16)
Type Telechat Review
Team General Area Review Team (Gen-ART) (genart)
Deadline 2014-08-05
Requested 2014-07-24
Authors Ravi Valmikam, Rajeev Koodli
I-D last updated 2014-08-07
Completed reviews Genart Last Call review of -08 by Peter E. Yee (diff)
Genart Telechat review of -09 by Peter E. Yee (diff)
Secdir Last Call review of -08 by Leif Johansson (diff)
Assignment Reviewer Peter E. Yee
State Completed
Request Telechat review on draft-ietf-netext-wifi-epc-eap-attributes by General Area Review Team (Gen-ART) Assigned
Reviewed revision 09 (document currently at 16)
Result Ready w/nits
Completed 2014-08-07
review-ietf-netext-wifi-epc-eap-attributes-09-genart-telechat-yee-2014-08-07-00
I am the assigned Gen-ART reviewer for this draft. For background on
Gen-ART, please see the FAQ at
<

http://wiki.tools.ietf.org/area/gen/trac/wiki/GenArtfaq>

Please wait for direction from your document shepherd or AD before posting a
new version of the draft.

Document: draft-ietf-netext-wifi-epc-eap-attributes-08
Reviewer: Peter Yee
Review Date: August-6-2014
IETF LC End Date: June-24-2014
IESG Telechat date: August-7-2014

Summary: This draft is basically ready for publication as an Informational
RFC, but has some nits that should be fixed before publication. [Ready with
nits.]

Most of my concerns from the LC review have been addressed very nicely.
Thank you!  Remaining concerns are:

General:

Leave URLs out of RFC references; they aren't included in the current
reference format.  (RFC 5415, for example.)

Nits:

Page 9, Section 5.2, 2nd paragraph, last sentence: I made a suggestion on
adding "value" after "Type field" in my previous review.  That seems to have
been reflected as inserting "value" between "Type" and "field".  I still
stand by my previous recommendation.

Please undo this previously made suggestion:  Page 3, paragraph after the
bullet points, 1st sentence: replace "and" before "EAP-SIM" with a comma.
	After further consideration, I believe my suggestion was in error.

Page 5, Section 3.2, 2nd paragraph, 2nd sentence: insert "an" before the
first occurrence of "MN".

Page 12, Section 6, 1st paragraph, 2nd sentence: change the "a" before "AAA"
to "an", unless the expected reading is "Triple A" or similar.

Page 12, Section 6, 2nd paragraph, 2nd sentence: insert "and" before
"AT_MN_SERIAL-ID".

Page 12, Section 6, 3rd paragraph, 2nd sentence: change "for" to "to".

Page 14, EPC reference: delete the semicolon. Delete the superfluous quote
and comma following the date.

Page 14, GPRS reference: Delete the superfluous quote and comma following
the date.  I'm also not sure why you dropped the "Stage 2" from the document
title since it does appear on the 3GPP site.